Animate window control changes in TouchView

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

Index: chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc b/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
index 19d916148cd326f012dea84d4b5fa3150ac6025a..04c4d6576d3374380756ae1f12d4352158b26d98 100644
--- a/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
@@ -4,6 +4,8 @@
 
 #include "chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.h"
 
+#include <algorithm>

[flackr, 2014/05/08 19:02:21]

not used?

[jonross, 2014/05/08 21:15:40]

Cpplint, used outside my changed.

+
 #include "ash/ash_switches.h"
 #include "ash/frame/caption_buttons/frame_caption_button_container_view.h"
 #include "ash/frame/default_header_painter.h"
@@ -377,6 +379,18 @@ gfx::ImageSkia BrowserNonClientFrameViewAsh::GetFaviconForTabIconView() {
 }
 
 ///////////////////////////////////////////////////////////////////////////////
+// views::View:
+
+void
+BrowserNonClientFrameViewAsh::ChildPreferredSizeChanged(views::View* child) {
+  if (child != caption_button_container_)

[flackr, 2014/05/08 19:02:21]

Also comment why.

+    return;
+  InvalidateLayout();
+  frame()->client_view()->InvalidateLayout();
+  frame()->GetRootView()->Layout();

[flackr, 2014/05/08 19:02:21]

I'm not quite sure what invalidating the layouts and doing an immediate layout
is necessary for.

[jonross, 2014/05/08 21:15:40]

That was recommended when I implemented the Maximize Mode observing in this
class.

However it does not affect this functionality.

+}
+
+///////////////////////////////////////////////////////////////////////////////
 // BrowserNonClientFrameViewAsh, private:
 
 int BrowserNonClientFrameViewAsh::GetTabStripLeftInset() const {