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

Side by Side Diff: chrome/browser/metrics/metrics_reporting_scheduler.h

Issue 8536054: Migrate several instances of NewRunnableMethod to base::Bind in browser/metrics (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_METRICS_REPORTING_SCHEDULER_H_ 5 #ifndef CHROME_BROWSER_METRICS_METRICS_REPORTING_SCHEDULER_H_
6 #define CHROME_BROWSER_METRICS_METRICS_REPORTING_SCHEDULER_H_ 6 #define CHROME_BROWSER_METRICS_METRICS_REPORTING_SCHEDULER_H_
7 #pragma once 7 #pragma once
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
11 #include "base/task.h" 11 #include "base/memory/weak_ptr.h"
12 #include "base/time.h" 12 #include "base/time.h"
13 13
14 // Scheduler task to drive a MetricsService object's uploading. 14 // Scheduler task to drive a MetricsService object's uploading.
15 class MetricsReportingScheduler { 15 class MetricsReportingScheduler {
16 public: 16 public:
17 explicit MetricsReportingScheduler(const base::Closure& upload_callback); 17 explicit MetricsReportingScheduler(const base::Closure& upload_callback);
18 ~MetricsReportingScheduler(); 18 ~MetricsReportingScheduler();
19 19
20 // Starts scheduling uploads. This in a no-op if the scheduler is already 20 // Starts scheduling uploads. This in a no-op if the scheduler is already
21 // running, so it is safe to call more than once. 21 // running, so it is safe to call more than once.
(...skipping 17 matching lines...) Expand all
39 // Schedules a future call to TriggerUpload if one isn't already pending. 39 // Schedules a future call to TriggerUpload if one isn't already pending.
40 void ScheduleNextCallback(); 40 void ScheduleNextCallback();
41 41
42 // Increases the upload interval each time it's called, to handle the case 42 // Increases the upload interval each time it's called, to handle the case
43 // where the server is having issues. 43 // where the server is having issues.
44 void BackOffUploadInterval(); 44 void BackOffUploadInterval();
45 45
46 // The MetricsService method to call when uploading should happen. 46 // The MetricsService method to call when uploading should happen.
47 base::Closure upload_callback_; 47 base::Closure upload_callback_;
48 48
49 ScopedRunnableMethodFactory<MetricsReportingScheduler> upload_timer_factory_; 49 base::WeakPtrFactory<MetricsReportingScheduler> weak_ptr_factory_;
50 50
51 // The interval between being told an upload is done and starting the next 51 // The interval between being told an upload is done and starting the next
52 // upload. 52 // upload.
53 base::TimeDelta upload_interval_; 53 base::TimeDelta upload_interval_;
54 54
55 // Indicates that the scheduler is running (i.e., that Start has been called 55 // Indicates that the scheduler is running (i.e., that Start has been called
56 // more recently than Stop). 56 // more recently than Stop).
57 bool running_; 57 bool running_;
58 58
59 // Indicates that a timer for triggering the next upload has already been 59 // Indicates that a timer for triggering the next upload has already been
60 // started. 60 // started.
61 bool timer_pending_; 61 bool timer_pending_;
62 62
63 // Indicates that the last triggered upload hasn't resolved yet. 63 // Indicates that the last triggered upload hasn't resolved yet.
64 bool callback_pending_; 64 bool callback_pending_;
65 65
66 DISALLOW_COPY_AND_ASSIGN(MetricsReportingScheduler); 66 DISALLOW_COPY_AND_ASSIGN(MetricsReportingScheduler);
67 }; 67 };
68 68
69 #endif // CHROME_BROWSER_METRICS_METRICS_REPORTING_SCHEDULER_H_ 69 #endif // CHROME_BROWSER_METRICS_METRICS_REPORTING_SCHEDULER_H_
OLDNEW
« no previous file with comments | « chrome/browser/metrics/histogram_synchronizer.cc ('k') | chrome/browser/metrics/metrics_reporting_scheduler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698