[mus+ash] Fixes minimum browser window size (store / retrieve kPreferredSize property)

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

Index: chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc
diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc b/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc
index 191623f6bf01fc928474c4ff4af2bebeab841b3e..a8994ce9dfa72fa47dc87cd6b8e498a68c81af57 100644
--- a/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc
+++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc
@@ -336,7 +336,16 @@ gfx::Size BrowserNonClientFrameViewMus::GetMinimumSize() const {
         std::max(min_width, min_tabstrip_width + GetTabStripLeftInset() +
                                 GetTabStripRightInset());
   }
-  return gfx::Size(min_width, min_client_view_size.height());
+  gfx::Size min_size(min_width, min_client_view_size.height());
+
+  aura::Window* frame_window = frame()->GetNativeWindow();
+  const gfx::Size* min_window_size =
+      frame_window->GetProperty(aura::client::kMinimumSize);
+  if (!min_window_size || *min_window_size != min_size) {
+    frame_window->SetProperty(aura::client::kMinimumSize,
+                              new gfx::Size(min_size));
+  }
+  return min_size;

[sadrul, 2017/06/01 17:14:20]

This is the only part that I am a bit worried about. Specifically, this
information in ash can be stale.

The information the minimum-size seems to depend on are:
 a. Minimum size of the client_view.
 b. frame_values().
 c. tab-strip visibility.
 d. Minimum width of tabstrip.
 e. left-inset of the tabstrip.
 f. right-inset of the tabstrip.

I think b, c, d, e never change after BrowserNonClientFrameViewMus has been
initialized.

f depends on the profile-switcher icon, but I believe we do not use that
chromeos. So f should also never change after BNCFVMus has been initialized.

So the remaining unknown is (a). I guess the client_view is a BrowserView? Looks
like its minimum size is determined by BrowserViewLayout, which takes the
minimum-size of the following views to determine the minimum-size of the
client_view:
 1. tab-strip.
 2. toolbar.
 3. bookmark bar.
 4. infobar.

Hm ... this is already getting too complex.

Essentially, what I think should happen is that the minimum-size should be set
during/right after BNCFVMus is initialized, and then any time something it
depends on changes. Maybe BrowserView can notify back when anything in 1-4
change?

[varkha, 2017/06/19 15:57:36]

Thanks for the ideas. Did you have something like this wiring in mind?

 }
 
 ///////////////////////////////////////////////////////////////////////////////