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

Side by Side Diff: sync/notifier/sync_notifier_factory_unittest.cc

Issue 10702074: Refactor sync-specific parts out of SyncNotifier/SyncNotifierObserver (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Minor cleanup 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 "sync/notifier/sync_notifier_factory.h" 5 #include "sync/notifier/sync_notifier_factory.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 notifier_options_.notification_method = notifier::NOTIFICATION_SERVER; 53 notifier_options_.notification_method = notifier::NOTIFICATION_SERVER;
54 SyncNotifierFactory factory( 54 SyncNotifierFactory factory(
55 notifier_options_, 55 notifier_options_,
56 "test client info", 56 "test client info",
57 base::WeakPtr<syncer::InvalidationStateTracker>()); 57 base::WeakPtr<syncer::InvalidationStateTracker>());
58 scoped_ptr<SyncNotifier> notifier(factory.CreateSyncNotifier()); 58 scoped_ptr<SyncNotifier> notifier(factory.CreateSyncNotifier());
59 #if defined(OS_ANDROID) 59 #if defined(OS_ANDROID)
60 ASSERT_FALSE(notifier.get()); 60 ASSERT_FALSE(notifier.get());
61 #else 61 #else
62 ASSERT_TRUE(notifier.get()); 62 ASSERT_TRUE(notifier.get());
63 notifier->AddObserver(&mock_observer_); 63 notifier->AddHandler(&mock_observer_);
64 notifier->RemoveObserver(&mock_observer_); 64 notifier->RemoveHandler(&mock_observer_);
65 #endif 65 #endif
66 } 66 }
67 67
68 // Test basic creation of a P2PNotifier. 68 // Test basic creation of a P2PNotifier.
69 TEST_F(SyncNotifierFactoryTest, Basic_P2P) { 69 TEST_F(SyncNotifierFactoryTest, Basic_P2P) {
70 notifier_options_.notification_method = notifier::NOTIFICATION_P2P; 70 notifier_options_.notification_method = notifier::NOTIFICATION_P2P;
71 SyncNotifierFactory factory( 71 SyncNotifierFactory factory(
72 notifier_options_, 72 notifier_options_,
73 "test client info", 73 "test client info",
74 base::WeakPtr<syncer::InvalidationStateTracker>()); 74 base::WeakPtr<syncer::InvalidationStateTracker>());
75 scoped_ptr<SyncNotifier> notifier(factory.CreateSyncNotifier()); 75 scoped_ptr<SyncNotifier> notifier(factory.CreateSyncNotifier());
76 #if defined(OS_ANDROID) 76 #if defined(OS_ANDROID)
77 ASSERT_FALSE(notifier.get()); 77 ASSERT_FALSE(notifier.get());
78 #else 78 #else
79 ASSERT_TRUE(notifier.get()); 79 ASSERT_TRUE(notifier.get());
80 notifier->AddObserver(&mock_observer_); 80 notifier->AddHandler(&mock_observer_);
81 notifier->RemoveObserver(&mock_observer_); 81 notifier->RemoveHandler(&mock_observer_);
82 #endif 82 #endif
83 } 83 }
84 84
85 } // namespace 85 } // namespace
86 } // namespace syncer 86 } // namespace syncer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698