Add ongoing events to net-export log when logging starts

chrome/browser/io_thread.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_IO_THREAD_H_
 #define CHROME_BROWSER_IO_THREAD_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 206 matching lines @@
   // Allows overriding Globals in tests where IOThread::Init() and
   // IOThread::CleanUp() are not called.  This allows for injecting mocks into
   // IOThread global objects.
   void SetGlobalsForTesting(Globals* globals);
 
   net_log::ChromeNetLog* net_log();
 
   // 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
   // by it are deleted from the cache.
   void ClearHostCache(
       const base::Callback<bool(const std::string&)>& host_filter);
 
   const net::HttpNetworkSession::Params& NetworkSessionParams() const;
 
   // Dynamically disables QUIC for HttpNetworkSessions owned by io_thread, and
@@ skipping 150 matching lines @@
 
   // These member variables are initialized by a task posted to the IO thread,
   // which gets posted by calling certain member functions of IOThread.
   std::unique_ptr<net::ProxyConfigService> system_proxy_config_service_;
 
   std::unique_ptr<PrefProxyConfigTracker> pref_proxy_config_tracker_;
 
   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_;
 
   // True if HTTP/0.9 is allowed on non-default ports by policy.
   bool http_09_on_non_default_ports_enabled_;
 
   const base::TimeTicks creation_time_;
 
   base::WeakPtrFactory<IOThread> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(IOThread);
 };
 
 #endif  // CHROME_BROWSER_IO_THREAD_H_