Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(105)

Unified Diff: components/quirks/quirks_client.h

Issue 1528963002: Quirks Client for downloading and providing display profiles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix ash_unittests Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..e2a236042086b120f826f23fdc2deec0e181c140
--- /dev/null
+++ b/components/quirks/quirks_client.h
@@ -0,0 +1,85 @@
+// 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 "net/base/backoff_entry.h"
+#include "net/url_request/url_fetcher_delegate.h"
+
+namespace quirks {
+
+class QuirksManager;
+
+// TODO: Can this path reference disappear during the callback?
stevenjb 2016/02/29 20:22:29 No. If a reference is bound to a callback using Bi
Greg Levin 2016/03/02 00:00:26 Acknowledged, thanks!
+using RequestFinishedCallback =
+ base::Callback<void(const base::FilePath&, bool)>;
+
+// Handles downloading icc and other display data files from Quirks Server.
+class QuirksClient : public net::URLFetcherDelegate {
+ public:
+ enum RequestReason {
+ FIRST, // Device's first server request.
+ RETRY, // Has server file been added since last check?
+ UPDATE // Has server file been updated since last check?
+ };
+
+ QuirksClient(int64_t product_id,
+ const RequestFinishedCallback& on_request_finished,
+ QuirksManager* manager);
+ ~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_;
+
+ // Reason for current server call.
+ RequestReason request_reason_;
+
+ // 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_

Powered by Google App Engine
This is Rietveld 408576698