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

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

Issue 239093007: Update Windows UI on system color changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make Widget the observer, address additional comments. 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_service_aurax11.cc
diff --git a/chrome/browser/themes/theme_service_aurax11.cc b/chrome/browser/themes/theme_service_aurax11.cc
index 7da0336070fa5bc8861cfa8d210314a9dfb097ef..b9313d5bee4a23fe1cd759db428989d98171a32d 100644
--- a/chrome/browser/themes/theme_service_aurax11.cc
+++ b/chrome/browser/themes/theme_service_aurax11.cc
@@ -64,17 +64,9 @@ NativeThemeX11::~NativeThemeX11() {}
} // namespace
-ThemeServiceAuraX11::ThemeServiceAuraX11() {
- views::LinuxUI* linux_ui = views::LinuxUI::instance();
- if (linux_ui)
- linux_ui->AddNativeThemeChangeObserver(this);
-}
+ThemeServiceAuraX11::ThemeServiceAuraX11() {}
-ThemeServiceAuraX11::~ThemeServiceAuraX11() {
- views::LinuxUI* linux_ui = views::LinuxUI::instance();
- if (linux_ui)
- linux_ui->RemoveNativeThemeChangeObserver(this);
-}
+ThemeServiceAuraX11::~ThemeServiceAuraX11() {}
bool ThemeServiceAuraX11::ShouldInitWithNativeTheme() const {
return profile()->GetPrefs()->GetBoolean(prefs::kUsesSystemTheme);
@@ -93,8 +85,3 @@ bool ThemeServiceAuraX11::UsingNativeTheme() const {
return theme_supplier &&
theme_supplier->get_theme_type() == CustomThemeSupplier::NATIVE_X11;
}
-
-void ThemeServiceAuraX11::OnNativeThemeChanged() {
- if (UsingNativeTheme())
- NotifyThemeChanged();
-}

Powered by Google App Engine
This is Rietveld 408576698