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

Unified Diff: ui/message_center/views/custom_notification_view.cc

Issue 2676473004: Update min/max notification height for N (Closed)
Patch Set: Update min/max notification height for N Created 3 years, 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/message_center/views/custom_notification_view.cc
diff --git a/ui/message_center/views/custom_notification_view.cc b/ui/message_center/views/custom_notification_view.cc
index 91b894673e90f13db892db6b1a47cf40f8714a49..e009bb856b3f1e529e99abc2db62bc99a8965fe5 100644
--- a/ui/message_center/views/custom_notification_view.cc
+++ b/ui/message_center/views/custom_notification_view.cc
@@ -81,7 +81,8 @@ gfx::Size CustomNotificationView::GetPreferredSize() const {
const int contents_width = kNotificationWidth - insets.width();
const int contents_height = contents_view_->GetHeightForWidth(contents_width);
- constexpr int kMaxContentHeight = 256;
+ // This is union of max/min height for M (256-64) and N (284-92).
+ constexpr int kMaxContentHeight = 284;
constexpr int kMinContentHeight = 64;
return gfx::Size(kNotificationWidth,
std::max(kMinContentHeight + insets.height(),
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698