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

Unified Diff: chrome/test/live_sync/live_themes_sync_test.cc

Issue 7599019: Allow sync integration tests to operate on multiple datatypes: Apps, Extensions, Themes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CR feedback Created 9 years, 4 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/test/live_sync/live_themes_sync_test.cc
diff --git a/chrome/test/live_sync/live_themes_sync_test.cc b/chrome/test/live_sync/live_themes_sync_test.cc
deleted file mode 100644
index 125ea75d49832226ee54ad7354be21f170b9d532..0000000000000000000000000000000000000000
--- a/chrome/test/live_sync/live_themes_sync_test.cc
+++ /dev/null
@@ -1,81 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/test/live_sync/live_themes_sync_test.h"
-
-#include "base/logging.h"
-#include "base/string_number_conversions.h"
-#include "chrome/browser/themes/theme_service.h"
-#include "chrome/browser/themes/theme_service_factory.h"
-#include "chrome/common/extensions/extension.h"
-
-namespace {
-
-// Make a name to pass to an extension helper.
-std::string MakeName(int index) {
- return "faketheme" + base::IntToString(index);
-}
-
-ThemeService* GetThemeService(Profile* profile) {
- return ThemeServiceFactory::GetForProfile(profile);
-}
-
-} // namespace
-
-LiveThemesSyncTest::LiveThemesSyncTest(TestType test_type)
- : LiveSyncTest(test_type) {}
-
-LiveThemesSyncTest::~LiveThemesSyncTest() {}
-
-bool LiveThemesSyncTest::SetupClients() {
- if (!LiveSyncTest::SetupClients())
- return false;
-
- extension_helper_.Setup(this);
- return true;
-}
-
-std::string LiveThemesSyncTest::GetCustomTheme(int index) const {
- return extension_helper_.NameToId(MakeName(index));
-}
-
-std::string LiveThemesSyncTest::GetThemeID(Profile* profile) const {
- return GetThemeService(profile)->GetThemeID();
-}
-
-bool LiveThemesSyncTest::UsingCustomTheme(Profile* profile) const {
- return GetThemeID(profile) != ThemeService::kDefaultThemeID;
-}
-
-bool LiveThemesSyncTest::UsingDefaultTheme(Profile* profile) const {
- return GetThemeService(profile)->UsingDefaultTheme();
-}
-
-bool LiveThemesSyncTest::UsingNativeTheme(Profile* profile) const {
- return GetThemeService(profile)->UsingNativeTheme();
-}
-
-bool LiveThemesSyncTest::ThemeIsPendingInstall(
- Profile* profile, const std::string& id) const {
- return extension_helper_.IsExtensionPendingInstallForSync(profile, id);
-}
-
-bool LiveThemesSyncTest::HasOrWillHaveCustomTheme(
- Profile* profile, const std::string& id) const {
- return (GetThemeID(profile) == id) || ThemeIsPendingInstall(profile, id);
-}
-
-void LiveThemesSyncTest::UseCustomTheme(Profile* profile, int index) {
- extension_helper_.InstallExtension(
- profile, MakeName(index), Extension::TYPE_THEME);
-}
-
-void LiveThemesSyncTest::UseDefaultTheme(Profile* profile) {
- GetThemeService(profile)->UseDefaultTheme();
-}
-
-void LiveThemesSyncTest::UseNativeTheme(Profile* profile) {
- // TODO(akalin): Fix this inconsistent naming in the theme service.
- GetThemeService(profile)->SetNativeTheme();
-}

Powered by Google App Engine
This is Rietveld 408576698