View: Resizing a layer in RTL now correctly repositions its children.

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 <map>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/rand_util.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
@@ skipping 2914 matching lines @@
   // sure the bounds are correct.
   View* v2 = new View;
   v2->SetBounds(50, 10, 30, 10);
   EXPECT_FALSE(v2->layer());
   view->AddChildView(v2);
   v2->SetPaintToLayer(true);
   EXPECT_EQ(gfx::Rect(content_width - 80, 10, 30, 10),
             v2->layer()->bounds());
   gfx::Rect l2bounds = v2->layer()->bounds();
 
+  // Now attach a paints-to-layer View as a child view of |v1|.
+  View* v3 = new View;
+  v3->SetPaintToLayer(true);
+  v3->SetBounds(3, 5, 6, 4);
+  EXPECT_EQ(gfx::Rect(3, 5, 6, 4),
+            v3->layer()->bounds());
+  v1->AddChildView(v3);
+  // Check that |v3| now has RTL-appropriate bounds.
+  EXPECT_EQ(gfx::Rect(11, 5, 6, 4),
+            v3->layer()->bounds());
+  gfx::Rect l3bounds = v3->layer()->bounds();
+
   view->SetPaintToLayer(true);
   EXPECT_EQ(l1bounds, v1->layer()->bounds());
   EXPECT_EQ(l2bounds, v2->layer()->bounds());
+  EXPECT_EQ(l3bounds, v3->layer()->bounds());
 
   // Move one of the views. Make sure the layer is positioned correctly
   // afterwards.
   v1->SetBounds(v1->x() - 5, v1->y(), v1->width(), v1->height());
   l1bounds.set_x(l1bounds.x() + 5);
   EXPECT_EQ(l1bounds, v1->layer()->bounds());
 
   view->SetPaintToLayer(false);
   EXPECT_EQ(l1bounds, v1->layer()->bounds());
   EXPECT_EQ(l2bounds, v2->layer()->bounds());
+  EXPECT_EQ(l3bounds, v3->layer()->bounds());
 
   // Move a view again.
   v2->SetBounds(v2->x() + 5, v2->y(), v2->width(), v2->height());
   l2bounds.set_x(l2bounds.x() - 5);
   EXPECT_EQ(l2bounds, v2->layer()->bounds());
 
+  // Resize |v1|. Make sure that |v3|'s layer has been moved correctly to
+  // RTL-appropriate bounds.
+  v1->SetSize(gfx::Size(30, 10));
+  EXPECT_EQ(gfx::Rect(21, 5, 6, 4),
+            v3->layer()->bounds());
+
   // Reset locale.
   base::i18n::SetICUDefaultLocale(locale);
 }
 
 // Makes sure a transform persists after toggling the visibility.
 TEST_F(ViewLayerTest, ToggleVisibilityWithTransform) {
   View* view = new View;
   gfx::Transform transform;
   transform.Scale(2.0, 2.0);
   view->SetTransform(transform);
@@ skipping 724 matching lines @@
   // notification.
   TestView* test_view_child_2 = new TestView();
   test_view->AddChildView(test_view_child_2);
   EXPECT_TRUE(test_view_child_2->native_theme_);
   EXPECT_EQ(widget->GetNativeTheme(), test_view_child_2->native_theme_);
 
   widget->CloseNow();
 }
 
 }  // namespace views