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

Unified Diff: sync/notifier/sync_system_resources.h

Issue 26594002: Chrome sync: Put WeakPtrFactory members at the end (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge (see prior patch for try jobs) Created 7 years, 2 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: sync/notifier/sync_system_resources.h
diff --git a/sync/notifier/sync_system_resources.h b/sync/notifier/sync_system_resources.h
index e333bb24f948ed6fd7a48320325b5324e69dd72b..b14d352bd1f1c817ae41b6f9b8219342f6fc53f6 100644
--- a/sync/notifier/sync_system_resources.h
+++ b/sync/notifier/sync_system_resources.h
@@ -65,7 +65,9 @@ class SyncInvalidationScheduler : public invalidation::Scheduler {
invalidation::SystemResources* resources) OVERRIDE;
private:
- base::WeakPtrFactory<SyncInvalidationScheduler> weak_factory_;
+ // Runs the task, deletes it, and removes it from |posted_tasks_|.
+ void RunPostedTask(invalidation::Closure* task);
+
// Holds all posted tasks that have not yet been run.
std::set<invalidation::Closure*> posted_tasks_;
@@ -73,8 +75,7 @@ class SyncInvalidationScheduler : public invalidation::Scheduler {
bool is_started_;
bool is_stopped_;
- // Runs the task, deletes it, and removes it from |posted_tasks_|.
- void RunPostedTask(invalidation::Closure* task);
+ base::WeakPtrFactory<SyncInvalidationScheduler> weak_factory_;
};
class SyncStorage : public invalidation::Storage {

Powered by Google App Engine
This is Rietveld 408576698