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

Side by Side Diff: chrome/browser/sync/glue/session_change_processor.h

Issue 26594002: Chrome sync: Put WeakPtrFactory members at the end (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_SYNC_GLUE_SESSION_CHANGE_PROCESSOR_H_ 5 #ifndef CHROME_BROWSER_SYNC_GLUE_SESSION_CHANGE_PROCESSOR_H_
6 #define CHROME_BROWSER_SYNC_GLUE_SESSION_CHANGE_PROCESSOR_H_ 6 #define CHROME_BROWSER_SYNC_GLUE_SESSION_CHANGE_PROCESSOR_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 64
65 private: 65 private:
66 void OnNavigationBlocked(content::WebContents* web_contents); 66 void OnNavigationBlocked(content::WebContents* web_contents);
67 67
68 // Utility method to handle reassociation of tabs and windows. 68 // Utility method to handle reassociation of tabs and windows.
69 void ProcessModifiedTabs( 69 void ProcessModifiedTabs(
70 const std::vector<SyncedTabDelegate*>& modified_tabs); 70 const std::vector<SyncedTabDelegate*>& modified_tabs);
71 71
72 void StartObserving(); 72 void StartObserving();
73 73
74 base::WeakPtrFactory<SessionChangeProcessor> weak_ptr_factory_;
75
76 SessionModelAssociator* session_model_associator_; 74 SessionModelAssociator* session_model_associator_;
77 content::NotificationRegistrar notification_registrar_; 75 content::NotificationRegistrar notification_registrar_;
78 76
79 // Profile being synced. Non-null if |running()| is true. 77 // Profile being synced. Non-null if |running()| is true.
80 Profile* profile_; 78 Profile* profile_;
81 79
82 // To bypass some checks/codepaths not applicable in tests. 80 // To bypass some checks/codepaths not applicable in tests.
83 bool setup_for_test_; 81 bool setup_for_test_;
84 82
83 base::WeakPtrFactory<SessionChangeProcessor> weak_ptr_factory_;
84
85 DISALLOW_COPY_AND_ASSIGN(SessionChangeProcessor); 85 DISALLOW_COPY_AND_ASSIGN(SessionChangeProcessor);
86 }; 86 };
87 87
88 } // namespace browser_sync 88 } // namespace browser_sync
89 89
90 #endif // CHROME_BROWSER_SYNC_GLUE_SESSION_CHANGE_PROCESSOR_H_ 90 #endif // CHROME_BROWSER_SYNC_GLUE_SESSION_CHANGE_PROCESSOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698