Add ability for NetLogLogger to gather data from more than just NetLog

chrome/browser/extensions/api/log_private/log_private_api.h

Index: chrome/browser/extensions/api/log_private/log_private_api.h
diff --git a/chrome/browser/extensions/api/log_private/log_private_api.h b/chrome/browser/extensions/api/log_private/log_private_api.h
index 4fbeaf8cd136dbea89f6e3b75a4dd5485a88417f..a3314a0a9c2631b7930bc19477410e7d6e05961f 100644
--- a/chrome/browser/extensions/api/log_private/log_private_api.h
+++ b/chrome/browser/extensions/api/log_private/log_private_api.h
@@ -8,6 +8,7 @@
 #include <set>
 #include <string>
 
+#include "base/files/scoped_file.h"
 #include "base/scoped_observer.h"
 #include "chrome/browser/extensions/api/log_private/filter_handler.h"
 #include "chrome/browser/extensions/api/log_private/log_parser.h"
@@ -88,14 +89,12 @@ class LogPrivateAPI : public BrowserContextKeyedAPI,
   void PostPendingEntries();
   void AddEntriesOnUI(scoped_ptr<base::ListValue> value);
 
-  // Initializes a new instance of net::NetLogLogger and passes it back via
-  // |net_logger| param.
-  void InitializeNetLogger(const std::string& owner_extension_id,
-                           net::NetLogLogger** net_logger);
+  // Creates a file for use with a new instance of net::NetLogLogger.
+  void CreateFileForNetLogger(
+      const std::string& owner_extension_id, base::ScopedFILE* file);

[not at google - send to devlin, 2015/03/09 21:26:25]

FILE!!!!!!!!!

[mmenke, 2015/03/09 21:59:30]

The problem with that is it leaks the FILE* if the task doesn't run on the io
thread for some reason.  I suppose since we're using base::Unretained, we assume
that doesn't happen, or if it does, it only happens when the IO Thread shuts
down?  I'm not that strongly opposed to switching, if you want, but it doesn't
strike me as all that safe.

[not at google - send to devlin, 2015/03/09 22:03:38]

Oh sorry, that was supposed to be a joke. I'm using to seeing naming like
"ScopedFile" so when I see "ScopedFILE" it looks like somebody shouting "FILE"
(though I do understand the naming convention).

As for the code in question, if CreateFileForNetLogger takes ownership then it
makes sense to be Scoped, otherwise, less so - I was just considering my input
as a rubber stamp.

[mmenke, 2015/03/09 22:21:43]

Ahh, right.  I completely misunderstood.  :)  And yes, it takes ownership.

I kept on writing File everywhere when writing this CL, and shockingly, every
time I tried, I got a compile failure.  :(

 
   // Starts observing network events with a new |net_logger| instance.
-  void StartObservingNetEvents(IOThread* io_thread,
-                               net::NetLogLogger** net_logger);
+  void StartObservingNetEvents(IOThread* io_thread, base::ScopedFILE* file);
   void MaybeStartNetInternalLogging(const std::string& caller_extension_id,
                                     IOThread* io_thread,
                                     api::log_private::EventSink event_sink);