Client Hints

chrome/browser/net/client_hints.h

Index: chrome/browser/net/client_hints.h
diff --git a/chrome/browser/net/client_hints.h b/chrome/browser/net/client_hints.h
new file mode 100644
index 0000000000000000000000000000000000000000..837e9c7f67d2aa248928eda4e642404a5038bde0
--- /dev/null
+++ b/chrome/browser/net/client_hints.h
@@ -0,0 +1,42 @@
+// Copyright 2013 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_NET_CLIENT_HINTS_H_
+#define CHROME_BROWSER_NET_CLIENT_HINTS_H_
+
+#include <string>
+
+#include "base/memory/weak_ptr.h"
+
+// ClientHints is a repository in Chrome for information used
+// to create the Client-Hints request header. For more information, see:
+// https://github.com/igrigorik/http-client-hints/blob/master/draft-grigorik-http-client-hints-01.txt
+class ClientHints {
+ public:
+  ClientHints();
+  ~ClientHints();
+
+  void Init();
+
+  // Retrieves screen information for use on the IO thread.
+  bool RetrieveScreenInfo();
+
+  // Returns client hints pertaining to screen information. The format is:
+  // "dpr=x", where x is pixel ratio. Its value is the same as the value
+  // returned by the JavaScript command window.devicePixelRatio

[mmenke, 2013/09/25 16:26:19]

nit:  .

[bengr, 2013/09/25 17:25:52]

Done.

+  const std::string& GetString() const;

[mmenke, 2013/09/25 16:26:19]

If the new ClientHints spec uses multiple fields, instead of a single CH field
with multiple values, I think this function name is wrong.  It should be
GetDeviceHeaderRatio();  (I know I suggested GetString(), but that was under the
assumption the spec was correct).

[bengr, 2013/09/25 17:25:52]

Changed to GetDevicePixelRatioHeader.

+
+ private:
+  friend class ClientHintsTest;
+
+  void UpdateScreenInfo(const float* device_pixel_ratio_value);
+
+  std::string screen_hints_;
+
+  base::WeakPtrFactory<ClientHints> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(ClientHints);
+};
+
+#endif  // CHROME_BROWSER_NET_CLIENT_HINTS_H_