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

Unified Diff: chrome/browser/themes/theme_syncable_service.cc

Issue 244893004: Improve some naming (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge and ShouldInitWithSystemTheme Created 6 years, 8 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/themes/theme_syncable_service.cc
diff --git a/chrome/browser/themes/theme_syncable_service.cc b/chrome/browser/themes/theme_syncable_service.cc
index 671a01fb569646e9d64732de0ca7937d5d76931c..fe140519e2bfe540438366ab3a7882d2a88ec00f 100644
--- a/chrome/browser/themes/theme_syncable_service.cc
+++ b/chrome/browser/themes/theme_syncable_service.cc
@@ -243,8 +243,8 @@ void ThemeSyncableService::SetCurrentThemeFromThemeSpecifics(
extensions_service->CheckForUpdatesSoon();
}
} else if (theme_specifics.use_system_theme_by_default()) {
- DVLOG(1) << "Switch to use native theme";
- theme_service_->SetNativeTheme();
+ DVLOG(1) << "Switch to use system theme";
+ theme_service_->UseSystemTheme();
} else {
DVLOG(1) << "Switch to use default theme";
theme_service_->UseDefaultTheme();
@@ -269,7 +269,7 @@ bool ThemeSyncableService::GetThemeSpecificsFromCurrentTheme(
// On platform where system theme is different from default theme, set
// use_system_theme_by_default to true if system theme is used, false
// if default system theme is used. Otherwise restore it to value in sync.
- if (theme_service_->UsingNativeTheme()) {
+ if (theme_service_->UsingSystemTheme()) {
theme_specifics->set_use_system_theme_by_default(true);
} else if (theme_service_->UsingDefaultTheme()) {
theme_specifics->set_use_system_theme_by_default(false);

Powered by Google App Engine
This is Rietveld 408576698