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

Unified Diff: ui/views/widget/widget.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
« chrome/browser/ui/libgtk2ui/gtk2_border.cc ('K') | « ui/views/widget/widget.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/widget/widget.cc
diff --git a/ui/views/widget/widget.cc b/ui/views/widget/widget.cc
index a01f90a3e547e1ae978c76656c853a0c864e572b..ba3723d0b3ec966d07faa9bdaf9f6639ec25bc69 100644
--- a/ui/views/widget/widget.cc
+++ b/ui/views/widget/widget.cc
@@ -183,7 +183,8 @@ Widget::Widget()
last_mouse_event_was_move_(false),
auto_release_capture_(true),
root_layers_dirty_(false),
- movement_disabled_(false) {
+ movement_disabled_(false),
+ observer_manager_(this) {
}
Widget::~Widget() {
@@ -368,6 +369,7 @@ void Widget::Init(const InitParams& in_params) {
internal::NativeWidgetPrivate::IsMouseButtonDown();
}
native_widget_->InitNativeWidget(params);
+ observer_manager_.Add(GetNativeTheme());
if (RequiresNonClientView(params.type)) {
non_client_view_ = new NonClientView;
non_client_view_->SetFrameView(CreateNonClientFrameView());
@@ -1339,6 +1341,13 @@ View* Widget::GetFocusTraversableParentView() {
}
////////////////////////////////////////////////////////////////////////////////
+// Widget, ui::NativeThemeObserver implementation:
+
+void Widget::OnNativeThemeUpdate() {
Evan Stade 2014/04/24 01:14:52 I still think this should pass the calling native
msw 2014/04/24 18:32:00 Done; this is a good idea, thanks for pushing for
+ root_view_->PropagateNativeThemeChanged(GetNativeTheme());
+}
+
+////////////////////////////////////////////////////////////////////////////////
// Widget, protected:
internal::RootView* Widget::CreateRootView() {
« chrome/browser/ui/libgtk2ui/gtk2_border.cc ('K') | « ui/views/widget/widget.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698