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

Side by Side Diff: chrome/browser/sync/glue/bridged_sync_notifier.cc

Issue 10702074: Refactor sync-specific parts out of SyncNotifier/SyncNotifierObserver (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 8 years, 5 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 #include "chrome/browser/sync/glue/bridged_sync_notifier.h" 5 #include "chrome/browser/sync/glue/bridged_sync_notifier.h"
6 6
7 #include "chrome/browser/sync/glue/chrome_sync_notification_bridge.h" 7 #include "chrome/browser/sync/glue/chrome_sync_notification_bridge.h"
8 8
9 namespace browser_sync { 9 namespace browser_sync {
10 10
11 BridgedSyncNotifier::BridgedSyncNotifier( 11 BridgedSyncNotifier::BridgedSyncNotifier(
12 ChromeSyncNotificationBridge* bridge, 12 ChromeSyncNotificationBridge* bridge,
13 syncer::SyncNotifier* delegate) 13 syncer::SyncNotifier* delegate)
14 : bridge_(bridge), delegate_(delegate) { 14 : bridge_(bridge), delegate_(delegate) {
15 DCHECK(bridge_); 15 DCHECK(bridge_);
16 } 16 }
17 17
18 BridgedSyncNotifier::~BridgedSyncNotifier() { 18 BridgedSyncNotifier::~BridgedSyncNotifier() {
19 } 19 }
20 20
21 void BridgedSyncNotifier::AddObserver( 21 void BridgedSyncNotifier::AddHandler(
22 syncer::SyncNotifierObserver* observer) { 22 syncer::SyncNotifierObserver* observer) {
23 if (delegate_.get()) 23 if (delegate_.get())
24 delegate_->AddObserver(observer); 24 delegate_->AddHandler(observer);
25 bridge_->AddObserver(observer); 25 bridge_->AddObserver(observer);
26 } 26 }
27 27
28 void BridgedSyncNotifier::RemoveObserver( 28 void BridgedSyncNotifier::RemoveHandler(
29 syncer::SyncNotifierObserver* observer) { 29 syncer::SyncNotifierObserver* observer) {
30 bridge_->RemoveObserver(observer); 30 bridge_->RemoveObserver(observer);
31 if (delegate_.get()) 31 if (delegate_.get())
32 delegate_->RemoveObserver(observer); 32 delegate_->RemoveHandler(observer);
33 }
34
35 void BridgedSyncNotifier::ReloadHandlers() {
36 if (delegate_.get())
37 delegate_->ReloadHandlers();
33 } 38 }
34 39
35 void BridgedSyncNotifier::SetUniqueId(const std::string& unique_id) { 40 void BridgedSyncNotifier::SetUniqueId(const std::string& unique_id) {
36 if (delegate_.get()) 41 if (delegate_.get())
37 delegate_->SetUniqueId(unique_id); 42 delegate_->SetUniqueId(unique_id);
38 } 43 }
39 44
40 void BridgedSyncNotifier::SetStateDeprecated(const std::string& state) { 45 void BridgedSyncNotifier::SetStateDeprecated(const std::string& state) {
41 if (delegate_.get()) 46 if (delegate_.get())
42 delegate_->SetStateDeprecated(state); 47 delegate_->SetStateDeprecated(state);
43 } 48 }
44 49
45 void BridgedSyncNotifier::UpdateCredentials( 50 void BridgedSyncNotifier::UpdateCredentials(
46 const std::string& email, const std::string& token) { 51 const std::string& email, const std::string& token) {
47 if (delegate_.get()) 52 if (delegate_.get())
48 delegate_->UpdateCredentials(email, token); 53 delegate_->UpdateCredentials(email, token);
49 } 54 }
50 55
51 void BridgedSyncNotifier::UpdateEnabledTypes(
52 syncer::ModelTypeSet enabled_types) {
53 if (delegate_.get())
54 delegate_->UpdateEnabledTypes(enabled_types);
55 }
56
57 void BridgedSyncNotifier::SendNotification( 56 void BridgedSyncNotifier::SendNotification(
58 syncer::ModelTypeSet changed_types) { 57 syncer::ModelTypeSet changed_types) {
59 if (delegate_.get()) 58 if (delegate_.get())
60 delegate_->SendNotification(changed_types); 59 delegate_->SendNotification(changed_types);
61 } 60 }
62 61
63 } // namespace browser_sync 62 } // namespace browser_sync
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698