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

Unified Diff: chrome/browser/ui/views/harmony/chrome_layout_provider.cc

Issue 2968713003: Harmonize the find in page dialog. (Closed)
Patch Set: format fixes Created 3 years, 5 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/harmony/chrome_layout_provider.cc
diff --git a/chrome/browser/ui/views/harmony/chrome_layout_provider.cc b/chrome/browser/ui/views/harmony/chrome_layout_provider.cc
index 53e14c4479e462fcb38209dddaf264e517d682a6..55fbe609237e93778a32b080a1c83d2f468f98ff 100644
--- a/chrome/browser/ui/views/harmony/chrome_layout_provider.cc
+++ b/chrome/browser/ui/views/harmony/chrome_layout_provider.cc
@@ -45,6 +45,10 @@ int ChromeLayoutProvider::GetDistanceMetric(int metric) const {
return 20;
case DISTANCE_UNRELATED_CONTROL_VERTICAL_LARGE:
return 30;
+ case DISTANCE_TOAST_CONTROL_VERTICAL:
+ return 8;
+ case DISTANCE_TOAST_LABEL_VERTICAL:
+ return 12;
default:
return views::LayoutProvider::GetDistanceMetric(metric);
}
@@ -58,6 +62,16 @@ const views::TypographyProvider& ChromeLayoutProvider::GetTypographyProvider()
return legacy_provider;
}
+gfx::Insets ChromeLayoutProvider::GetInsetsMetric(int metric) const {
+ DCHECK_LT(metric, views::VIEWS_INSETS_MAX);
Peter Kasting 2017/07/15 07:38:36 No need for this line, the base class checks it
ananta 2017/07/17 19:11:04 Done.
+ switch (metric) {
+ case InsetsMetric::INSETS_TOAST:
+ return gfx::Insets(0, 8);
+ default:
+ return views::LayoutProvider::GetInsetsMetric(metric);
+ }
+}
+
views::GridLayout::Alignment
ChromeLayoutProvider::GetControlLabelGridAlignment() const {
return views::GridLayout::TRAILING;

Powered by Google App Engine
This is Rietveld 408576698