Added Net logging to FileStream.

net/base/file_stream.h

Index: net/base/file_stream.h
diff --git a/net/base/file_stream.h b/net/base/file_stream.h
index f2a8f7f4f94701027ed5a2a08b822dec83491922..142f806ca89cb689431fb7883ab7ff38dd73f7cf 100644
--- a/net/base/file_stream.h
+++ b/net/base/file_stream.h
@@ -15,6 +15,7 @@
 #include "base/platform_file.h"
 #include "net/base/completion_callback.h"
 #include "net/base/net_export.h"
+#include "net/base/net_log.h"

[Randy Smith (Not in Mondays), 2012/01/30 23:29:13]

Why can't this just be forward references?

[ahendrickson, 2012/01/31 20:12:40]

Because we have a BoundNetLog member variable.

 
 class FilePath;
 
@@ -32,6 +33,9 @@ class NET_EXPORT FileStream {
  public:
   FileStream();
 
+  // Create a new BoundNetLog and attach it to the stream.

[rvargas (doing something else), 2012/01/30 23:11:38]

nit: "Constructs a FileStream..." or similar

[ahendrickson, 2012/01/31 20:12:40]

Done.

+  FileStream(net::NetLog* log);

[rvargas (doing something else), 2012/01/30 23:11:38]

Is the idea to remove the default constructor once the users are updated?

[ahendrickson, 2012/01/31 20:12:40]

Removed the default constructor, based on this and other comments.

+
   // Construct a FileStream with an existing file handle and opening flags.
   // |file| is valid file handle.
   // |flags| is a bitfield of base::PlatformFileFlags when the file handle was
@@ -136,6 +140,9 @@ class NET_EXPORT FileStream {
   // Turns on UMA error statistics gathering.
   void EnableErrorStatistics();
 
+  // Sets the source reference for net-internals logging.
+  void SetBoundNetLogSource(const net::BoundNetLog& log);
+
  private:
   class AsyncContext;
   friend class AsyncContext;
@@ -149,6 +156,7 @@ class NET_EXPORT FileStream {
   int open_flags_;
   bool auto_closed_;
   bool record_uma_;
+  net::BoundNetLog net_log_;
 
   DISALLOW_COPY_AND_ASSIGN(FileStream);
 };