Update Windows UI on system color changes.

ui/views/widget/root_view.cc

 // Copyright (c) 2012 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 "ui/views/widget/root_view.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/cursor/cursor.h"
 #include "ui/base/dragdrop/drag_drop_types.h"
 #include "ui/compositor/layer.h"
 #include "ui/events/event.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 #include "ui/gfx/canvas.h"
+#include "ui/native_theme/native_theme.h"
 #include "ui/views/focus/view_storage.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/view_targeter.h"
 #include "ui/views/views_switches.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_delegate.h"
 
 typedef ui::EventDispatchDetails DispatchDetails;
 
 namespace views {
@@ skipping 84 matching lines @@
       gesture_handler_(NULL),
       scroll_gesture_handler_(NULL),
       pre_dispatch_handler_(new internal::PreEventDispatchHandler(this)),
       focus_search_(this, false, false),
       focus_traversable_parent_(NULL),
       focus_traversable_parent_view_(NULL),
       event_dispatch_target_(NULL),
       old_dispatch_target_(NULL) {
   AddPreTargetHandler(pre_dispatch_handler_.get());
   SetEventTargeter(scoped_ptr<ui::EventTargeter>(new ViewTargeter()));
+  GetNativeTheme()->AddObserver(this);

[Evan Stade, 2014/04/22 17:43:44]

ScopedObserver?

[msw, 2014/04/22 19:48:58]

Eh, it doesn't seem worthwhile to add a class member to handle one
RemoveObserver call; what do you think?

[Evan Stade, 2014/04/22 21:09:18]

I always prefer scoping variables over doing stuff manually, which is error
prone.

[msw, 2014/04/23 00:34:04]

Done (in Widget).

 }
 
 RootView::~RootView() {
+  GetNativeTheme()->RemoveObserver(this);
   // If we have children remove them explicitly so to make sure a remove
   // notification is sent for each one of them.
   if (has_children())
     RemoveAllChildViews(true);
 }
 
 // Tree operations -------------------------------------------------------------
 
 void RootView::SetContentsView(View* contents_view) {
   DCHECK(contents_view && GetWidget()->native_widget()) <<
@@ skipping 360 matching lines @@
 void RootView::GetAccessibleState(ui::AXViewState* state) {
   state->name = widget_->widget_delegate()->GetAccessibleWindowTitle();
   state->role = widget_->widget_delegate()->GetAccessibleWindowRole();
 }
 
 void RootView::UpdateParentLayer() {
   if (layer())
     ReparentLayer(gfx::Vector2d(GetMirroredX(), y()), widget_->GetLayer());
 }
 
+void RootView::OnNativeThemeChange() {
+  PropagateNativeThemeChanged(GetNativeTheme());
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // RootView, protected:
 
 void RootView::DispatchGestureEvent(ui::GestureEvent* event) {
   if (gesture_handler_) {
     // |gesture_handler_| (or |scroll_gesture_handler_|) can be deleted during
     // processing.
     View* handler = scroll_gesture_handler_ &&
         (event->IsScrollGestureEvent() || event->IsFlingScrollEvent())  ?
             scroll_gesture_handler_ : gesture_handler_;
@@ skipping 261 matching lines @@
 
 #ifndef NDEBUG
   DCHECK(!event_dispatch_target_ || Contains(event_dispatch_target_));
 #endif
 
   return details;
 }
 
 }  // namespace internal
 }  // namespace views