App list: Fixed rendering issues upon opening the window in RTL mode.

ui/app_list/views/contents_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.
 
 #include "ui/app_list/views/contents_view.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 476 matching lines @@
     rect = GetContentsBounds();
     progress = 1;
   }
 
   // Notify the custom launcher page that the active page has changed.
   app_list_main_view_->view_delegate()->CustomLauncherPageAnimationChanged(
       progress);
 
   if (rect.IsEmpty())
     return;
+  // TODO(mgiuca): Temporary work-around for http://crbug.com/441962 and
+  // http://crbug.com/446978. This will first be called while ContentsView is
+  // 0x0, which means that the child views will be positioned incorrectly in RTL
+  // mode (the position is based on the parent's size). When the parent is later
+  // resized, the children are not repositioned due to http://crbug.com/446407.
+  // Therefore, we must not position the children until the parent is the
+  // correct size.
+  // NOTE: There is a similar hack in AppsGridView::CalculateIdealBounds; both
+  // should be removed once http://crbug.com/446407 is resolved.
+  if (GetContentsBounds().IsEmpty())
+    return;
 
   gfx::Rect offscreen_target(rect);
   offscreen_target.set_x(-rect.width());
 
   int current_page = GetActivePageIndex();
 
   for (int i = 0; i < view_model_->view_size(); ++i) {
     view_model_->view_at(i)
         ->SetBoundsRect(i == current_page ? rect : offscreen_target);
   }
@@ skipping 27 matching lines @@
 }
 
 void ContentsView::TransitionStarted() {
 }
 
 void ContentsView::TransitionChanged() {
   UpdatePageBounds();
 }
 
 }  // namespace app_list