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

Side by Side Diff: chrome/browser/themes/theme_syncable_service_unittest.cc

Issue 15701022: [Sync] Add support for sync Persistence Errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move bookmark change into separate patch Created 7 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/themes/theme_syncable_service.h" 5 #include "chrome/browser/themes/theme_syncable_service.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/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 487 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 498
499 // Change current theme to custom theme and notify theme_sync_service_. 499 // Change current theme to custom theme and notify theme_sync_service_.
500 // No change is output because sync has stopped. 500 // No change is output because sync has stopped.
501 fake_theme_service_->SetTheme(theme_extension_.get()); 501 fake_theme_service_->SetTheme(theme_extension_.get());
502 theme_sync_service_->OnThemeChange(); 502 theme_sync_service_->OnThemeChange();
503 EXPECT_EQ(0u, change_list.size()); 503 EXPECT_EQ(0u, change_list.size());
504 504
505 // ProcessSyncChanges() should return error when sync has stopped. 505 // ProcessSyncChanges() should return error when sync has stopped.
506 error = theme_sync_service_->ProcessSyncChanges(FROM_HERE, change_list); 506 error = theme_sync_service_->ProcessSyncChanges(FROM_HERE, change_list);
507 EXPECT_TRUE(error.IsSet()); 507 EXPECT_TRUE(error.IsSet());
508 EXPECT_EQ(syncer::THEMES, error.type()); 508 EXPECT_EQ(syncer::THEMES, error.model_type());
509 EXPECT_EQ("Theme syncable service is not started.", 509 EXPECT_EQ("datatype error was encountered: Theme syncable service is not "
510 "started.",
510 error.message()); 511 error.message());
511 } 512 }
512 513
513 TEST_F(ThemeSyncableServiceTest, RestoreSystemThemeBitWhenChangeToCustomTheme) { 514 TEST_F(ThemeSyncableServiceTest, RestoreSystemThemeBitWhenChangeToCustomTheme) {
514 syncer::SyncChangeList change_list; 515 syncer::SyncChangeList change_list;
515 static_cast<FakeSyncChangeProcessor*>(fake_change_processor_.get())-> 516 static_cast<FakeSyncChangeProcessor*>(fake_change_processor_.get())->
516 SetChangeOutput(&change_list); 517 SetChangeOutput(&change_list);
517 518
518 // Initialize to use system theme. 519 // Initialize to use system theme.
519 fake_theme_service_->UseDefaultTheme(); 520 fake_theme_service_->UseDefaultTheme();
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
611 612
612 TEST_F(PolicyInstalledThemeTest, InstallThemeByPolicy) { 613 TEST_F(PolicyInstalledThemeTest, InstallThemeByPolicy) {
613 // Set up theme service to use custom theme that was installed by policy. 614 // Set up theme service to use custom theme that was installed by policy.
614 fake_theme_service_->SetTheme(theme_extension_.get()); 615 fake_theme_service_->SetTheme(theme_extension_.get());
615 616
616 syncer::SyncDataList data_list = 617 syncer::SyncDataList data_list =
617 theme_sync_service_->GetAllSyncData(syncer::THEMES); 618 theme_sync_service_->GetAllSyncData(syncer::THEMES);
618 619
619 ASSERT_EQ(0u, data_list.size()); 620 ASSERT_EQ(0u, data_list.size());
620 } 621 }
OLDNEW
« no previous file with comments | « chrome/browser/themes/theme_syncable_service.cc ('k') | chrome/browser/webdata/autocomplete_syncable_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698