ChromeOS: Implement periodic timezone refresh on geolocation data.

chromeos/timezone/timezone_resolver.h

+// Copyright 2014 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 CHROMEOS_TIMEZONE_TIMEZONE_RESOLVER_H_
+#define CHROMEOS_TIMEZONE_TIMEZONE_RESOLVER_H_
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/threading/thread_checker.h"
+#include "chromeos/chromeos_export.h"
+#include "url/gurl.h"
+
+class PrefRegistrySimple;
+
+namespace net {
+class URLRequestContextGetter;
+}
+
+namespace chromeos {
+
+struct TimeZoneResponseData;
+
+// This class implements periodic timezone synchronization.
+class CHROMEOS_EXPORT TimeZoneResolver {
+ public:
+  class TimeZoneResolverImpl;
+
+  // This callback will be called when new timezone arrives.
+  typedef base::Callback<void(const TimeZoneResponseData*)>
+      ApplyTimeZoneCallback;
+
+  // chromeos::DelayNetworkCall cannot be used directly due to link
+  // restrictions.
+  typedef base::Callback<void(const base::Closure&)> DelayNetworkCallClosure;
+
+  TimeZoneResolver(net::URLRequestContextGetter* context,
+                   const GURL& url,
+                   const ApplyTimeZoneCallback& apply_timezone,
+                   const DelayNetworkCallClosure& delay_network_call);
+  ~TimeZoneResolver();
+
+  // Starts periodic timezone refresh.
+  void Start();
+
+  // Cancels current request and stops periodic timezone refresh.
+  void Stop();
+
+ private:
+  net::URLRequestContextGetter* const context_;
+  const GURL url_;
+
+  const ApplyTimeZoneCallback apply_timezone_;
+  const DelayNetworkCallClosure delay_network_call_;
+
+  scoped_ptr<TimeZoneResolverImpl> implementation_;
+
+  base::ThreadChecker thread_checker_;
+
+  DISALLOW_COPY_AND_ASSIGN(TimeZoneResolver);
+};
+
+// Expose internal fuctions for testing.
+unsigned int CHROMEOS_EXPORT

[Bernhard Bauer, 2015/01/16 09:25:05]

Per the style guide, we usually use just plain int for numbers, or a fixed-size
type if that is important (like uint64_t).

[Alexander Alekseev, 2015/01/16 20:42:45]

Done.

+MaxRequestsCountForIntervalForTesting(const double interval_seconds);

[Bernhard Bauer, 2015/01/16 09:25:05]

Could you make these methods static class methods?

[Alexander Alekseev, 2015/01/16 20:42:44]

Done.

+
+unsigned int CHROMEOS_EXPORT
+IntervalForNextRequestForTesting(const unsigned int requests);
+
+}  // namespace chromeos
+
+#endif  // CHROMEOS_TIMEZONE_TIMEZONE_RESOLVER_H_