Quirks Client for downloading and providing display profiles

components/quirks/quirks_client.h

Index: components/quirks/quirks_client.h
diff --git a/components/quirks/quirks_client.h b/components/quirks/quirks_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..90926647bae757f8cc2e6bb8ffb40f74a97aff17
--- /dev/null
+++ b/components/quirks/quirks_client.h
@@ -0,0 +1,71 @@
+// Copyright 2016 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 COMPONENTS_QUIRKS_QUIRKS_CLIENT_H_
+#define COMPONENTS_QUIRKS_QUIRKS_CLIENT_H_
+
+#include "base/files/file_path.h"
+#include "base/macros.h"
+#include "base/timer/timer.h"
+#include "components/quirks/quirks_manager.h"
+#include "net/base/backoff_entry.h"
+#include "net/url_request/url_fetcher_delegate.h"
+
+namespace quirks {
+
+// Handles downloading icc and other display data files from Quirks Server.
+class QuirksClient : public net::URLFetcherDelegate {
+ public:
+  QuirksClient(int64_t product_id,
+               const RequestFinishedCallback& on_request_finished,
+               QuirksManager* manager);

[Bernhard Bauer, 2016/03/17 17:42:25]

You could forward-declare QuirksManager in the header instead of including it.

[Greg Levin, 2016/03/18 00:35:46]

Done.
(I included the header here for RequestFinishedCallback, but "using" it in both
headers seems to work.)

+  ~QuirksClient() override;
+
+  void StartDownload();
+
+  int64_t product_id() { return product_id_; }
+
+ private:
+  // net::URLFetcherDelegate:
+  void OnURLFetchComplete(const net::URLFetcher* source) override;
+
+  // Send callback and tell manager to delete |this|.
+  void Shutdown(bool success);
+
+  // Schedules retry.
+  void Retry();
+
+  // Translate json with base64-encoded data (|result|) into raw |data|.
+  bool ParseResult(const std::string& result, std::string* data);
+
+  // ID of display to request from Quirks Server.
+  const int64_t product_id_;
+
+  // Callback supplied by caller.
+  const RequestFinishedCallback on_request_finished_;
+
+  // Weak pointer owned by manager, guaranteed to outlive this client object.
+  QuirksManager* manager_;
+
+  // Full path to icc file.
+  const base::FilePath icc_path_;
+
+  // This fetcher is used to download icc file.
+  scoped_ptr<net::URLFetcher> url_fetcher_;
+
+  // Pending retry.
+  base::OneShotTimer request_scheduled_;
+
+  // Controls exponential backoff of time between server checks.
+  net::BackoffEntry backoff_entry_;
+
+  // Factory for callbacks.
+  base::WeakPtrFactory<QuirksClient> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(QuirksClient);
+};
+
+}  // namespace quirks
+
+#endif  // COMPONENTS_QUIRKS_QUIRKS_CLIENT_H_