Add ongoing events to net-export log when logging starts

chrome/browser/io_thread.h

Index: chrome/browser/io_thread.h
diff --git a/chrome/browser/io_thread.h b/chrome/browser/io_thread.h
index 5bf834ad3917b3f93a9337726cec701d151a9a7e..c579e365b043da773611042cb56db24b9f050f2c 100644
--- a/chrome/browser/io_thread.h
+++ b/chrome/browser/io_thread.h
@@ -224,9 +224,14 @@ class IOThread : public content::BrowserThreadDelegate {
   // Handles changing to On The Record mode, discarding confidential data.
   void ChangedToOnTheRecord();
 
-  // Returns a getter for the URLRequestContext.  Only called on the UI thread.
+  // Returns a getter for the system URLRequestContext. Only called on the UI
+  // thread.
   net::URLRequestContextGetter* system_url_request_context_getter();
 
+  // Returns a getter for the proxy script fetcher URLRequestContext. Only
+  // called on the UI thread.
+  net::URLRequestContextGetter* proxy_script_fetcher_context_getter();

[eroman, 2017/02/21 22:14:23]

This is only used by NetExportUI.

Instead of exposing it as API in IOThread can you move the corresponding
implementation into net_export_ui.cc?

io_thread.{h,cc} is already a big ball of spaghetti, so I would like to avoid
adding more stuff into it which isn't core.

In this case the need for a URLRequestContextGetter is only a constraint for the
NetExportUI so we can isolate the ugliness to that client.

[wangyix1, 2017/02/22 01:28:18]

Done.

+
   // Clears the host cache. Intended to be used to prevent exposing recently
   // visited sites on about:net-internals/#dns and about:dns pages.  Must be
   // called on the IO thread. If |host_filter| is not null, only hosts matched
@@ -397,6 +402,9 @@ class IOThread : public content::BrowserThreadDelegate {
   scoped_refptr<net::URLRequestContextGetter>
       system_url_request_context_getter_;
 
+  scoped_refptr<net::URLRequestContextGetter>
+      proxy_script_fetcher_context_getter_;
+
   // True if QUIC is allowed by policy.
   bool is_quic_allowed_by_policy_;