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

Side by Side Diff: chrome/browser/ui/webui/settings/people_handler.h

Issue 2657673004: Add shutdown notification and service refs to SyncServiceObserver. (Closed)
Patch Set: Chromeos fix Created 3 years, 10 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_ 5 #ifndef CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_
6 #define CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_ 6 #define CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 void FocusUI() override; 112 void FocusUI() override;
113 113
114 // SigninManagerBase::Observer implementation. 114 // SigninManagerBase::Observer implementation.
115 void GoogleSigninSucceeded(const std::string& account_id, 115 void GoogleSigninSucceeded(const std::string& account_id,
116 const std::string& username, 116 const std::string& username,
117 const std::string& password) override; 117 const std::string& password) override;
118 void GoogleSignedOut(const std::string& account_id, 118 void GoogleSignedOut(const std::string& account_id,
119 const std::string& username) override; 119 const std::string& username) override;
120 120
121 // syncer::SyncServiceObserver implementation. 121 // syncer::SyncServiceObserver implementation.
122 void OnStateChanged() override; 122 void OnStateChanged(syncer::SyncService* sync) override;
123 123
124 // Returns a newly created dictionary with a number of properties that 124 // Returns a newly created dictionary with a number of properties that
125 // correspond to the status of sync. 125 // correspond to the status of sync.
126 std::unique_ptr<base::DictionaryValue> GetSyncStatusDictionary(); 126 std::unique_ptr<base::DictionaryValue> GetSyncStatusDictionary();
127 127
128 // Helper routine that gets the ProfileSyncService associated with the parent 128 // Helper routine that gets the ProfileSyncService associated with the parent
129 // profile. 129 // profile.
130 browser_sync::ProfileSyncService* GetSyncService() const; 130 browser_sync::ProfileSyncService* GetSyncService() const;
131 131
132 // Returns the LoginUIService for the parent profile. 132 // Returns the LoginUIService for the parent profile.
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 ScopedObserver<SigninManagerBase, PeopleHandler> signin_observer_; 200 ScopedObserver<SigninManagerBase, PeopleHandler> signin_observer_;
201 ScopedObserver<browser_sync::ProfileSyncService, PeopleHandler> 201 ScopedObserver<browser_sync::ProfileSyncService, PeopleHandler>
202 sync_service_observer_; 202 sync_service_observer_;
203 203
204 DISALLOW_COPY_AND_ASSIGN(PeopleHandler); 204 DISALLOW_COPY_AND_ASSIGN(PeopleHandler);
205 }; 205 };
206 206
207 } // namespace settings 207 } // namespace settings
208 208
209 #endif // CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_ 209 #endif // CHROME_BROWSER_UI_WEBUI_SETTINGS_PEOPLE_HANDLER_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/options/sync_setup_handler_unittest.cc ('k') | chrome/browser/ui/webui/settings/people_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698