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

ui/views/view.cc

Index: ui/views/view.cc
diff --git a/ui/views/view.cc b/ui/views/view.cc
index 0fc755972d6d57de055a7ea9d31fa11733eaf13b..f4d20dd5838d2f20dd50019bd5b592b62d03ea57 100644
--- a/ui/views/view.cc
+++ b/ui/views/view.cc
@@ -725,7 +725,9 @@ void View::SchedulePaintInRect(const gfx::Rect& rect) {
 
   if (layer()) {
     layer()->SchedulePaint(rect);
-  } else if (parent_) {
+  }
+
+  if (parent_ && (!layer() || !layer()->fills_bounds_opaquely())) {

[jonross, 2015/11/18 16:55:45]

This will trigger for all schedule paints on a non-opaque layer.

We really need it for the initial transition where the parent is dirty.
Subsequently parent and child should track when they get dirty.

[bruthig, 2015/11/18 17:02:48]

Agreed, I just want to make sure the following scenario is addressed then.

If the non-opaque child layer changes what is drawn, and more specifically
changes some previously opaque pixles to now be transparent, then what mechanism
causes the newly transparent pixels to be 'filled' in with the parents pixels?
In other words, the parent never became dirty here, but it does indeed need to
be painted again.

[bruthig, 2015/11/18 17:46:42]

Discussed offline.

     // Translate the requested paint rect to the parent's coordinate system
     // then pass this notification up to the parent.
     parent_->SchedulePaintInRect(ConvertRectToParent(rect));
@@ -1384,8 +1386,14 @@ void View::OnPaintBorder(gfx::Canvas* canvas) {
 
 void View::SetFillsBoundsOpaquely(bool fills_bounds_opaquely) {
   // This method should not have the side-effect of creating the layer.
-  if (layer())
+  if (layer()) {
     layer()->SetFillsBoundsOpaquely(fills_bounds_opaquely);
+    if (!fills_bounds_opaquely && parent_) {
+      // Translate the requested paint rect to the parent's coordinate system
+      // then pass this notification up to the parent.
+      parent_->SchedulePaintInRect(ConvertRectToParent(bounds_));
+    }
+  }
 }
 
 gfx::Vector2d View::CalculateOffsetToAncestorWithLayer(