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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/views/device_chooser_content_view.cc

Issue 2750063002: views: implement dialog width snapping (Closed)
Patch Set: width -> min_width and improve tests Created 3 years, 8 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
« no previous file with comments | « chrome/browser/ui/views/content_setting_bubble_contents.cc ('k') | chrome/browser/ui/views/extensions/chooser_dialog_view.cc » ('j') | chrome/browser/ui/views/harmony/layout_delegate.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/device_chooser_content_view.cc
diff --git a/chrome/browser/ui/views/device_chooser_content_view.cc b/chrome/browser/ui/views/device_chooser_content_view.cc
index c37bf3663c9a8caefa9d86dbcf4c1a24a380e908..2bd78a6728d6b8dffaf478908e5cf15b30ea79ad 100644
--- a/chrome/browser/ui/views/device_chooser_content_view.cc
+++ b/chrome/browser/ui/views/device_chooser_content_view.cc
@@ -112,13 +112,7 @@ void DeviceChooserContentView::Layout() {
}
gfx::Size DeviceChooserContentView::GetPreferredSize() const {
- constexpr int kHeight = 320;
- constexpr int kDefaultWidth = 402;
- int width = LayoutDelegate::Get()->GetDialogPreferredWidth(
- LayoutDelegate::DialogWidth::MEDIUM);
- if (!width)
- width = kDefaultWidth;
- return gfx::Size(width, kHeight);
+ return gfx::Size(402, 320);
}
int DeviceChooserContentView::RowCount() {
« no previous file with comments | « chrome/browser/ui/views/content_setting_bubble_contents.cc ('k') | chrome/browser/ui/views/extensions/chooser_dialog_view.cc » ('j') | chrome/browser/ui/views/harmony/layout_delegate.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698