Use correctly mirrored points for ideal bounds of launcher item icons.

ash/wm/panels/panel_window_resizer_unittest.cc

Index: ash/wm/panels/panel_window_resizer_unittest.cc
diff --git a/ash/wm/panels/panel_window_resizer_unittest.cc b/ash/wm/panels/panel_window_resizer_unittest.cc
index 9039de294e22eefde2493e347e5a489cceea65be..def04f5a9d6f48d844c619993933f6108cc68ae9 100644
--- a/ash/wm/panels/panel_window_resizer_unittest.cc
+++ b/ash/wm/panels/panel_window_resizer_unittest.cc
@@ -193,6 +193,21 @@ class PanelWindowResizerTest : public test::AshTestBase {
   DISALLOW_COPY_AND_ASSIGN(PanelWindowResizerTest);
 };
 
+class PanelWindowResizerRTLTest : public PanelWindowResizerTest {
+ public:
+  PanelWindowResizerRTLTest() {}
+  virtual ~PanelWindowResizerRTLTest() {}
+
+  virtual void SetUp() OVERRIDE {
+    base::i18n::SetICUDefaultLocale("ar");
+    PanelWindowResizerTest::SetUp();
+    ASSERT_TRUE(base::i18n::IsRTL());
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(PanelWindowResizerRTLTest);
+};
+

[varkha, 2013/05/03 17:48:14]

RTL could be a test parameter set up with INSTANTIATE_TEST_CASE_P. Then all the
tests would run twice - with and without RTL. You can then tune which tests we
want to run with RTL and which not. I think this will improve the coverage.

 // Verifies a window can be dragged from the panel and detached and then
 // reattached.
 TEST_F(PanelWindowResizerTest, PanelDetachReattachBottom) {
@@ -414,6 +429,10 @@ TEST_F(PanelWindowResizerTest, DragReordersPanelsHorizontal) {
   DragAlongShelfReorder(-1, 0);
 }
 
+TEST_F(PanelWindowResizerRTLTest, DragReordersPanelsHorizontal) {
+  DragAlongShelfReorder(1, 0);
+}
+
 TEST_F(PanelWindowResizerTest, DragReordersPanelsVertical) {
   ash::Shell* shell = ash::Shell::GetInstance();
   shell->SetShelfAlignment(SHELF_ALIGNMENT_LEFT, shell->GetPrimaryRootWindow());