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

Unified Diff: chrome/browser/extensions/theme_installed_infobar_delegate.cc

Issue 244893004: Improve some naming (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync 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/extensions/theme_installed_infobar_delegate.cc
diff --git a/chrome/browser/extensions/theme_installed_infobar_delegate.cc b/chrome/browser/extensions/theme_installed_infobar_delegate.cc
index 2b9b673b142a9a0ecf55456b83207dbb7a7b768d..e3f5c8c41fa8a1f93a3fadbe339688c08b1fa83a 100644
--- a/chrome/browser/extensions/theme_installed_infobar_delegate.cc
+++ b/chrome/browser/extensions/theme_installed_infobar_delegate.cc
@@ -28,7 +28,7 @@ void ThemeInstalledInfoBarDelegate::Create(
const extensions::Extension* new_theme,
Profile* profile,
const std::string& previous_theme_id,
- bool previous_using_native_theme) {
+ bool previous_using_system_theme) {
DCHECK(new_theme);
if (!new_theme->is_theme())
return;
@@ -52,7 +52,7 @@ void ThemeInstalledInfoBarDelegate::Create(
ConfirmInfoBarDelegate::CreateInfoBar(scoped_ptr<ConfirmInfoBarDelegate>(
new ThemeInstalledInfoBarDelegate(
profile->GetExtensionService(), theme_service, new_theme,
- previous_theme_id, previous_using_native_theme))));
+ previous_theme_id, previous_using_system_theme))));
// If there's a previous theme infobar, just replace that instead of adding a
// new one.
@@ -82,14 +82,14 @@ ThemeInstalledInfoBarDelegate::ThemeInstalledInfoBarDelegate(
ThemeService* theme_service,
const extensions::Extension* new_theme,
const std::string& previous_theme_id,
- bool previous_using_native_theme)
+ bool previous_using_system_theme)
: ConfirmInfoBarDelegate(),
extension_service_(extension_service),
theme_service_(theme_service),
name_(new_theme->name()),
theme_id_(new_theme->id()),
previous_theme_id_(previous_theme_id),
- previous_using_native_theme_(previous_using_native_theme) {
+ previous_using_system_theme_(previous_using_system_theme) {
registrar_.Add(this, chrome::NOTIFICATION_BROWSER_THEME_CHANGED,
content::Source<ThemeService>(theme_service_));
}
@@ -142,8 +142,8 @@ bool ThemeInstalledInfoBarDelegate::Cancel() {
}
}
- if (previous_using_native_theme_)
- theme_service_->SetNativeTheme();
+ if (previous_using_system_theme_)
+ theme_service_->UseSystemTheme();
else
theme_service_->UseDefaultTheme();
return false; // The theme change will close us.

Powered by Google App Engine
This is Rietveld 408576698