Updated Client-Hints patch

chrome/browser/net/chrome_network_delegate.h

Index: chrome/browser/net/chrome_network_delegate.h
diff --git a/chrome/browser/net/chrome_network_delegate.h b/chrome/browser/net/chrome_network_delegate.h
index e9bf978f1c8b1e0468dc47e4f0e1f881f87e8254..366c0f372a88357e38682b8206583695e77ac63b 100644
--- a/chrome/browser/net/chrome_network_delegate.h
+++ b/chrome/browser/net/chrome_network_delegate.h
@@ -11,6 +11,7 @@
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
+#include "chrome/browser/net/client_hints.h"

[mmenke, 2013/09/09 16:55:30]

This should be forward declared instead of included in the header.  Should go
right before "class ConnectInterceptor" below, to be in alphabetical order.

 #include "net/base/network_delegate.h"
 
 class CookieSettings;
@@ -90,6 +91,9 @@ class ChromeNetworkDelegate : public net::NetworkDelegate {
     force_google_safe_search_ = force_google_safe_search;
   }
 
+  // Adds the Client Hints header to HTTP requests.
+  void set_enable_client_hints();

[mmenke, 2013/09/09 16:55:30]

Only inlined functions should use this naming style.  And generally if you call
something that's not inlined, the calling function shouldn't be inlined either.

So this should be "SetEnableClientHints()".  I actually think the "set" is
redundant, and "EnableClientHints()" would be a little clearer.

+
   // Causes |OnCanThrottleRequest| to always return false, for all
   // instances of this object.
   static void NeverThrottleRequests();
@@ -207,6 +211,8 @@ class ChromeNetworkDelegate : public net::NetworkDelegate {
   // Total original size of all content before it was transferred.
   int64 original_content_length_;
 
+  scoped_ptr<ClientHints> client_hints_;
+
   DISALLOW_COPY_AND_ASSIGN(ChromeNetworkDelegate);
 };