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

ui/views/view_unittest.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/view.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <memory>
@@ skipping 5005 matching lines @@
 TEST_F(ViewObserverTest, ChildViewReordered) {
   std::unique_ptr<View> view = NewView();
   std::unique_ptr<View> child_view = NewView();
   std::unique_ptr<View> child_view2 = NewView();
   view->AddChildView(child_view.get());
   view->AddChildView(child_view2.get());
   view->ReorderChildView(child_view2.get(), 0);
   EXPECT_EQ(child_view2.get(), view_reordered());
 }
 
+// Validates that if a child of a ScrollView adds a layer, then a layer
+// is added to the ScrollView's viewport.
+TEST_F(ViewObserverTest, ScrollViewChildAddLayerTest) {
+  ScrollView* scroll_view = new ScrollView();

[sky, 2017/04/18 15:35:48]

Declare this on the stack, otherwise you're going to leak it.

[ananta, 2017/04/18 22:22:39]

Changed to unique_ptr

+  scroll_view->SetContents(new View());
+  EXPECT_TRUE(scroll_view->contents_viewport()->layer() == nullptr);

[sky, 2017/04/18 15:35:48]

EXPECT_FALSE(scroll_view->contents_viewport()->layer())

[ananta, 2017/04/18 22:22:39]

Done.

+
+  std::unique_ptr<View> child_view = NewView();
+  scroll_view->AddChildView(child_view.get());
+  child_view->SetBounds(0, 0, 100, 100);
+  child_view->SetPaintToLayer(ui::LAYER_TEXTURED);
+
+  EXPECT_TRUE(scroll_view->contents_viewport()->layer() != nullptr);

[sky, 2017/04/18 15:35:48]

EXPECT_TRUE(scroll_view->contents_viewport()->layer())

[ananta, 2017/04/18 22:22:39]

Done.

+  scroll_view->RemoveChildView(child_view.get());
+}
+
 }  // namespace views