Revert of Moved parent_->SchedulePaint() to Views::SetFillsBoundsOpaquely() and Views::ScheduleP

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 1749 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(GetLocalBounds()));
-  }
-}
-
 // 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 293 matching lines @@
 
   // The new layer needs to be ordered in the layer tree according
   // to the view tree. Children of this layer were added in order
   // in UpdateParentLayers().
   if (parent())
     parent()->ReorderLayers();
 
   Widget* widget = GetWidget();
   if (widget)
     widget->UpdateRootLayers();
-
-  // Before having its own Layer, this View may have painted in to a Layer owned
-  // by an ancestor View. Scheduling a paint on the parent View will erase this
-  // View's painting effects on the ancestor View's Layer.
-  // (See crbug.com/551492)
-  SchedulePaintOnParent();
 }
 
 void View::UpdateParentLayers() {
   // Attach all top-level un-parented layers.
   if (layer() && !layer()->parent()) {
     UpdateParentLayer();
   } else {
     for (int i = 0, count = child_count(); i < count; ++i)
       child_at(i)->UpdateParentLayers();
   }
@@ skipping 295 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