Moved parent_->SchedulePaint() to Views::SetFillsBoundsOpaquely() and Views::SchedulePaint().

ui/views/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.
 
 #define _USE_MATH_DEFINES // For VC++ to get M_PI. This has to be first.
 
 #include "ui/views/view.h"
 
 #include <algorithm>
 #include <cmath>
@@ skipping 443 matching lines @@
   }
 }
 
 void View::SetPaintToLayer(bool paint_to_layer) {
   if (paint_to_layer_ == paint_to_layer)
     return;
 
   paint_to_layer_ = paint_to_layer;
   if (paint_to_layer_ && !layer()) {
     CreateLayer();
+    SchedulePaintOnParent();

[sky, 2015/11/19 22:31:48]

Why does this case necessitate a paint on the parent? A comment would help here.

[bruthig, 2015/12/22 22:32:40]

Done.

   } else if (!paint_to_layer_ && layer()) {
     DestroyLayer();
   }
 }
 
 // RTL positioning -------------------------------------------------------------
 
 gfx::Rect View::GetMirroredBounds() const {
   gfx::Rect bounds(bounds_);
   bounds.set_x(GetMirroredX());
@@ skipping 903 matching lines @@
                  "width", canvas->sk_canvas()->getDevice()->width(),
                  "height", canvas->sk_canvas()->getDevice()->height());
     border_->Paint(*this, canvas);
   }
 }
 
 // Accelerated Painting --------------------------------------------------------
 
 void View::SetFillsBoundsOpaquely(bool fills_bounds_opaquely) {
   // This method should not have the side-effect of creating the layer.
-  if (layer())
+  if (layer()) {
+    bool schedule_paint_on_parent =

[sky, 2015/11/19 22:31:48]

Please add an early return to make this code easier to reason about, eg:

if (!layer() || layer()->fills_bounds_opaquely() == fills_...)
  return;

[bruthig, 2015/12/22 22:32:40]

Done.

+        !fills_bounds_opaquely && layer()->fills_bounds_opaquely();
     layer()->SetFillsBoundsOpaquely(fills_bounds_opaquely);
+    if (schedule_paint_on_parent)
+      SchedulePaintOnParent();
+  }
 }
 
 gfx::Vector2d View::CalculateOffsetToAncestorWithLayer(
     ui::Layer** layer_parent) {
   if (layer()) {
     if (layer_parent)
       *layer_parent = layer();
     return gfx::Vector2d();
   }
   if (!parent_)
@@ skipping 327 matching lines @@
     SchedulePaint();
   } else if (parent_ && type == SCHEDULE_PAINT_SIZE_SAME) {
     // The compositor doesn't Draw() until something on screen changes, so
     // if our position changes but nothing is being animated on screen, then
     // tell the compositor to redraw the scene. We know layer() exists due to
     // the above if clause.
     layer()->ScheduleDraw();
   }
 }
 
+void View::SchedulePaintOnParent() {
+  if (parent_) {
+    // Translate the requested paint rect to the parent's coordinate system
+    // then pass this notification up to the parent.
+    parent_->SchedulePaintInRect(ConvertRectToParent(bounds_));
+  }
+}
+
 // Tree operations -------------------------------------------------------------
 
 void View::DoRemoveChildView(View* view,
                              bool update_focus_cycle,
                              bool update_tool_tip,
                              bool delete_removed_view,
                              View* new_parent) {
   DCHECK(view);
 
   const Views::iterator i(std::find(children_.begin(), children_.end(), view));
@@ skipping 608 matching lines @@
   // Message the RootView to do the drag and drop. That way if we're removed
   // the RootView can detect it and avoid calling us back.
   gfx::Point widget_location(event.location());
   ConvertPointToWidget(this, &widget_location);
   widget->RunShellDrag(this, data, widget_location, drag_operations, source);
   // WARNING: we may have been deleted.
   return true;
 }
 
 }  // namespace views