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

Side by Side Diff: chrome/browser/metrics/variations/variations_request_scheduler_mobile.h

Issue 840813009: Enable strict-virtual-specifiers for iOS builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comments Created 5 years, 11 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 unified diff | Download patch
« no previous file with comments | « build/common.gypi ('k') | chrome/browser/policy/cloud/user_policy_signin_service_mobile.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_REQUEST_SCHEDULER_MOBILE_H_ 5 #ifndef CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_REQUEST_SCHEDULER_MOBILE_H_
6 #define CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_REQUEST_SCHEDULER_MOBILE_H_ 6 #define CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_REQUEST_SCHEDULER_MOBILE_H_
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "chrome/browser/metrics/variations/variations_request_scheduler.h" 9 #include "chrome/browser/metrics/variations/variations_request_scheduler.h"
10 10
11 class PrefService; 11 class PrefService;
12 12
13 namespace chrome_variations { 13 namespace chrome_variations {
14 14
15 // A specialized VariationsRequestScheduler that manages request cycles for 15 // A specialized VariationsRequestScheduler that manages request cycles for
16 // VariationsService on mobile platforms. 16 // VariationsService on mobile platforms.
17 class VariationsRequestSchedulerMobile : public VariationsRequestScheduler { 17 class VariationsRequestSchedulerMobile : public VariationsRequestScheduler {
18 public: 18 public:
19 // |task} is the closure to call when the scheduler deems ready. |local_state| 19 // |task} is the closure to call when the scheduler deems ready. |local_state|
20 // is the PrefService that contains the time of the last fetch. 20 // is the PrefService that contains the time of the last fetch.
21 explicit VariationsRequestSchedulerMobile(const base::Closure& task, 21 explicit VariationsRequestSchedulerMobile(const base::Closure& task,
22 PrefService* local_state); 22 PrefService* local_state);
23 virtual ~VariationsRequestSchedulerMobile(); 23 ~VariationsRequestSchedulerMobile() override;
24 24
25 // Base class overrides. 25 // Base class overrides.
26 virtual void Start() override; 26 void Start() override;
27 virtual void Reset() override; 27 void Reset() override;
28 virtual void OnAppEnterForeground() override; 28 void OnAppEnterForeground() override;
29 29
30 private: 30 private:
31 FRIEND_TEST_ALL_PREFIXES(VariationsRequestSchedulerMobileTest, 31 FRIEND_TEST_ALL_PREFIXES(VariationsRequestSchedulerMobileTest,
32 OnAppEnterForegroundNoRun); 32 OnAppEnterForegroundNoRun);
33 FRIEND_TEST_ALL_PREFIXES(VariationsRequestSchedulerMobileTest, 33 FRIEND_TEST_ALL_PREFIXES(VariationsRequestSchedulerMobileTest,
34 OnAppEnterForegroundRun); 34 OnAppEnterForegroundRun);
35 FRIEND_TEST_ALL_PREFIXES(VariationsRequestSchedulerMobileTest, 35 FRIEND_TEST_ALL_PREFIXES(VariationsRequestSchedulerMobileTest,
36 OnAppEnterForegroundOnStartup); 36 OnAppEnterForegroundOnStartup);
37 37
38 // The local state instance that provides the last fetch time. 38 // The local state instance that provides the last fetch time.
39 PrefService* local_state_; 39 PrefService* local_state_;
40 40
41 // Timer used for triggering a delayed fetch for ScheduleFetch(). 41 // Timer used for triggering a delayed fetch for ScheduleFetch().
42 base::OneShotTimer<VariationsRequestSchedulerMobile> schedule_fetch_timer_; 42 base::OneShotTimer<VariationsRequestSchedulerMobile> schedule_fetch_timer_;
43 43
44 // The time the last seed request was initiated. 44 // The time the last seed request was initiated.
45 base::Time last_request_time_; 45 base::Time last_request_time_;
46 46
47 DISALLOW_COPY_AND_ASSIGN(VariationsRequestSchedulerMobile); 47 DISALLOW_COPY_AND_ASSIGN(VariationsRequestSchedulerMobile);
48 }; 48 };
49 49
50 } // namespace chrome_variations 50 } // namespace chrome_variations
51 51
52 #endif // CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_REQUEST_SCHEDULER_MOBILE _H_ 52 #endif // CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_REQUEST_SCHEDULER_MOBILE _H_
OLDNEW
« no previous file with comments | « build/common.gypi ('k') | chrome/browser/policy/cloud/user_policy_signin_service_mobile.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698