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

Unified Diff: components/sync/driver/glue/browser_thread_model_worker_unittest.cc

Issue 2388673002: Revert of [Sync] Move //components/sync to the syncer namespace. (patchset #5 id:40001 of https://co (Closed)
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: components/sync/driver/glue/browser_thread_model_worker_unittest.cc
diff --git a/components/sync/driver/glue/browser_thread_model_worker_unittest.cc b/components/sync/driver/glue/browser_thread_model_worker_unittest.cc
index 7d4f02e1d514e1d669c477ae22a87838447d726e..883cae2b0570363e59966c6aa3c40f353668b02d 100644
--- a/components/sync/driver/glue/browser_thread_model_worker_unittest.cc
+++ b/components/sync/driver/glue/browser_thread_model_worker_unittest.cc
@@ -20,7 +20,7 @@
using base::Thread;
using base::TimeDelta;
-namespace syncer {
+namespace browser_sync {
namespace {
@@ -40,21 +40,21 @@ class SyncBrowserThreadModelWorkerTest : public testing::Test {
// DoWork hasn't executed within action_timeout().
void ScheduleWork() {
// We wait until the callback is done. So it is safe to use unretained.
- WorkCallback c = base::Bind(&SyncBrowserThreadModelWorkerTest::DoWork,
- base::Unretained(this));
+ syncer::WorkCallback c = base::Bind(
+ &SyncBrowserThreadModelWorkerTest::DoWork, base::Unretained(this));
timer()->Start(FROM_HERE, TestTimeouts::action_timeout(), this,
&SyncBrowserThreadModelWorkerTest::Timeout);
worker()->DoWorkAndWaitUntilDone(c);
}
// This is the work that will be scheduled to be done on the DB thread.
- SyncerError DoWork() {
+ syncer::SyncerError DoWork() {
EXPECT_TRUE(db_thread_.task_runner()->BelongsToCurrentThread());
timer_.Stop(); // Stop the failure timer so the test succeeds.
main_message_loop_.task_runner()->PostTask(
FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
did_do_work_ = true;
- return SYNCER_OK;
+ return syncer::SYNCER_OK;
}
// This will be called by the OneShotTimer and make the test fail unless
@@ -68,8 +68,8 @@ class SyncBrowserThreadModelWorkerTest : public testing::Test {
protected:
void SetUp() override {
db_thread_.Start();
- worker_ =
- new BrowserThreadModelWorker(db_thread_.task_runner(), GROUP_DB, NULL);
+ worker_ = new BrowserThreadModelWorker(db_thread_.task_runner(),
+ syncer::GROUP_DB, NULL);
}
virtual void Teardown() {
@@ -97,4 +97,4 @@ TEST_F(SyncBrowserThreadModelWorkerTest, DoesWorkOnDatabaseThread) {
} // namespace
-} // namespace syncer
+} // namespace browser_sync

Powered by Google App Engine
This is Rietveld 408576698