Conditionally call SchedulePaint() when activating and deactivating inactive rendering

ui/views/window/non_client_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/window/non_client_view.h"
 
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/hit_test.h"
 #include "ui/views/widget/root_view.h"
 #include "ui/views/widget/widget.h"
@@ skipping 176 matching lines @@
       return frame_view_->GetEventHandlerForPoint(point_in_child_coords);
   }
 
   return View::GetEventHandlerForPoint(point);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NonClientFrameView, public:
 
 void NonClientFrameView::SetInactiveRenderingDisabled(bool disable) {
-  // See comment in Widget::SetInactiveRenderingDisabled as to why we don't
-  // conditionally invoke ShouldPaintAsActiveChanged.
+  if (paint_as_active_ == disable)
+    return;
+
   paint_as_active_ = disable;
   ShouldPaintAsActiveChanged();
 }
 
 int NonClientFrameView::GetHTComponentForFrame(const gfx::Point& point,
                                                int top_resize_border_height,
                                                int resize_border_thickness,
                                                int top_resize_corner_height,
                                                int resize_corner_width,
                                                bool can_resize) {
@@ skipping 50 matching lines @@
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NonClientFrameView, protected:
 
 bool NonClientFrameView::ShouldPaintAsActive() const {
   return GetWidget()->IsActive() || paint_as_active_;
 }
 
 void NonClientFrameView::ShouldPaintAsActiveChanged() {
-  if (!paint_as_active_)
-    SchedulePaint();
+  SchedulePaint();
 }
 
 void NonClientFrameView::GetAccessibleState(ui::AccessibleViewState* state) {
   state->role = ui::AccessibilityTypes::ROLE_WINDOW;
 }
 
 std::string NonClientFrameView::GetClassName() const {
   return kViewClassName;
 }
 
 void NonClientFrameView::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   // Overridden to do nothing. The NonClientView manually calls Layout on the
   // FrameView when it is itself laid out, see comment in NonClientView::Layout.
 }
 
 }  // namespace views