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

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: Update tests Created 7 years, 6 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("Theme syncable service is not started.",
510 error.message()); 510 error.message());
511 } 511 }
512 512
513 TEST_F(ThemeSyncableServiceTest, RestoreSystemThemeBitWhenChangeToCustomTheme) { 513 TEST_F(ThemeSyncableServiceTest, RestoreSystemThemeBitWhenChangeToCustomTheme) {
514 syncer::SyncChangeList change_list; 514 syncer::SyncChangeList change_list;
515 static_cast<FakeSyncChangeProcessor*>(fake_change_processor_.get())-> 515 static_cast<FakeSyncChangeProcessor*>(fake_change_processor_.get())->
516 SetChangeOutput(&change_list); 516 SetChangeOutput(&change_list);
517 517
518 // Initialize to use system theme. 518 // Initialize to use system theme.
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
611 611
612 TEST_F(PolicyInstalledThemeTest, InstallThemeByPolicy) { 612 TEST_F(PolicyInstalledThemeTest, InstallThemeByPolicy) {
613 // Set up theme service to use custom theme that was installed by policy. 613 // Set up theme service to use custom theme that was installed by policy.
614 fake_theme_service_->SetTheme(theme_extension_.get()); 614 fake_theme_service_->SetTheme(theme_extension_.get());
615 615
616 syncer::SyncDataList data_list = 616 syncer::SyncDataList data_list =
617 theme_sync_service_->GetAllSyncData(syncer::THEMES); 617 theme_sync_service_->GetAllSyncData(syncer::THEMES);
618 618
619 ASSERT_EQ(0u, data_list.size()); 619 ASSERT_EQ(0u, data_list.size());
620 } 620 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698