Ensure that the focus ring in the bookmarks bar does not paint outside the parent view.

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 522 matching lines @@
   }
 }
 
 void View::SetPaintToLayer(ui::LayerType layer_type) {
   if (paint_to_layer_ && (layer()->type() == layer_type))
     return;
 
   DestroyLayer();
   CreateLayer(layer_type);
   paint_to_layer_ = true;
+
+  // Notify the parent chain about the layer change.
+  View* view = this;

[sky, 2017/04/17 15:24:01]

View* view = parent();
and then you don't need two conditionals on 546 (and 547 needs to change then).

[ananta, 2017/04/18 03:04:25]

Done.

+  while (view && view->parent()) {
+    view->parent()->OnChildLayerChanged(view);
+    view = view->parent();
+  }
 }
 
 void View::DestroyLayer() {

[sky, 2017/04/17 15:24:01]

DestroyLayer needs to call OnChildLayerChanged too.

[ananta, 2017/04/18 03:04:25]

Done.

   if (!paint_to_layer_)
     return;
 
   paint_to_layer_ = false;
   if (!layer())
     return;
 
   ui::Layer* new_parent = layer()->parent();
   std::vector<ui::Layer*> children = layer()->children();
   for (size_t i = 0; i < children.size(); ++i) {
@@ skipping 2088 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