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

Unified Diff: chrome/browser/sync/glue/chrome_sync_notification_bridge_unittest.cc

Issue 9512005: [Sync] Move BridgedSyncNotifier and ChromeSyncNotificationBridge to glue/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to head Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/glue/chrome_sync_notification_bridge_unittest.cc
diff --git a/chrome/browser/sync/notifier/chrome_sync_notification_bridge_unittest.cc b/chrome/browser/sync/glue/chrome_sync_notification_bridge_unittest.cc
similarity index 93%
rename from chrome/browser/sync/notifier/chrome_sync_notification_bridge_unittest.cc
rename to chrome/browser/sync/glue/chrome_sync_notification_bridge_unittest.cc
index ffc56f58439ae66dd2a2ec744bc211f26ea20de4..abd2c7bf5b80dab03b16c2bebb0404a789bedeec 100644
--- a/chrome/browser/sync/notifier/chrome_sync_notification_bridge_unittest.cc
+++ b/chrome/browser/sync/glue/chrome_sync_notification_bridge_unittest.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/sync/notifier/chrome_sync_notification_bridge.h"
+#include "chrome/browser/sync/glue/chrome_sync_notification_bridge.h"
#include "base/compiler_specific.h"
#include "base/memory/weak_ptr.h"
@@ -22,7 +22,7 @@
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
-namespace sync_notifier {
+namespace browser_sync {
namespace {
using ::testing::Mock;
@@ -37,7 +37,8 @@ using content::BrowserThread;
// Note: Because this object lives on the IO thread, we use a fake (vs a mock)
// so we don't have to worry about possible thread safety issues within
// GTest/GMock.
-class FakeSyncNotifierObserverIO : public SyncNotifierObserver {
+class FakeSyncNotifierObserverIO
+ : public sync_notifier::SyncNotifierObserver {
public:
FakeSyncNotifierObserverIO(
ChromeSyncNotificationBridge* bridge,
@@ -57,10 +58,10 @@ class FakeSyncNotifierObserverIO : public SyncNotifierObserver {
// SyncNotifierObserver implementation.
virtual void OnIncomingNotification(
const syncable::ModelTypePayloadMap& type_payloads,
- IncomingNotificationSource source) OVERRIDE {
+ sync_notifier::IncomingNotificationSource source) OVERRIDE {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
notification_count_++;
- if (source != LOCAL_NOTIFICATION) {
+ if (source != sync_notifier::LOCAL_NOTIFICATION) {
LOG(ERROR) << "Received notification with wrong source.";
received_improper_notification_ = true;
}
@@ -189,9 +190,10 @@ class ChromeSyncNotificationBridgeTest : public testing::Test {
TEST_F(ChromeSyncNotificationBridgeTest, Basic) {
syncable::ModelTypePayloadMap payload_map;
payload_map[syncable::SESSIONS] = "";
- StrictMock<MockSyncNotifierObserver> observer;
+ StrictMock<sync_notifier::MockSyncNotifierObserver> observer;
EXPECT_CALL(observer,
- OnIncomingNotification(payload_map, LOCAL_NOTIFICATION));
+ OnIncomingNotification(payload_map,
+ sync_notifier::LOCAL_NOTIFICATION));
bridge_.AddObserver(&observer);
TriggerRefreshNotification();
ui_loop_.RunAllPending();
@@ -210,4 +212,4 @@ TEST_F(ChromeSyncNotificationBridgeTest, BasicThreaded) {
}
} // namespace
-} // namespace sync_notifier
+} // namespace browser_sync
« no previous file with comments | « chrome/browser/sync/glue/chrome_sync_notification_bridge.cc ('k') | chrome/browser/sync/glue/sync_backend_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698