Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(986)

Unified Diff: chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc

Issue 2914793005: [mus+ash] Fixes minimum browser window size (store / retrieve kPreferredSize property) (Closed)
Patch Set: [mus+ash] Fixes minimum browser window size (nits / rebase) Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..3ebb3d056da37cb8cb8a6d7a5bbcc296368117fb 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
@@ -226,6 +226,17 @@ void BrowserNonClientFrameViewMus::UpdateClientArea() {
}
}
+void BrowserNonClientFrameViewMus::UpdateMinimumSize() {
+ gfx::Size min_size = GetMinimumSize();
+ aura::Window* frame_window = frame()->GetNativeWindow();
+ const gfx::Size* previous_min_size =
+ frame_window->GetProperty(aura::client::kMinimumSize);
+ if (!previous_min_size || *previous_min_size != min_size) {
+ frame_window->SetProperty(aura::client::kMinimumSize,
+ new gfx::Size(min_size));
+ }
+}
+
///////////////////////////////////////////////////////////////////////////////
// views::NonClientFrameView:
@@ -336,7 +347,8 @@ 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());
sky 2017/06/22 03:41:10 Convert back to old code as there is no reason for
varkha 2017/06/22 17:50:14 Done.
+ return min_size;
}
///////////////////////////////////////////////////////////////////////////////
« no previous file with comments | « chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.h ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698