Animate window control changes in TouchView

chrome/browser/ui/views/frame/browser_header_painter_ash.cc

Index: chrome/browser/ui/views/frame/browser_header_painter_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc b/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
index be2dbc75e27497608f0689bf925628f852811823..13cbd52792e6f43d66833b8eb0c24a3e115c3991 100644
--- a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
@@ -232,7 +232,6 @@ void BrowserHeaderPainterAsh::LayoutHeader() {
   painted_height_ = -1;
 
   UpdateCaptionButtonImages();
-  caption_button_container_->Layout();
 
   gfx::Size caption_button_container_size =
       caption_button_container_->GetPreferredSize();
@@ -241,6 +240,7 @@ void BrowserHeaderPainterAsh::LayoutHeader() {
       0,
       caption_button_container_size.width(),
       caption_button_container_size.height());
+  caption_button_container_->Layout();

[flackr, 2014/05/22 15:46:18]

Ditto, isn't there a risk of using an invalid preferred size to set the caption
button container position?

[jonross, 2014/05/22 20:36:04]

No, as preferred size reflects the current visibility. While layout is
restricted to the bounds set on the view.

On 2014/05/22 15:46:18, flackr wrote:

 
   if (window_icon_) {
     // Vertically center the window icon with respect to the caption button