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

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

Issue 2654323002: harmony: convert device chooser (Closed)
Patch Set: s/gfx::Size()// 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 | chrome/browser/ui/views/device_chooser_content_view.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/collected_cookies_views.cc
diff --git a/chrome/browser/ui/views/collected_cookies_views.cc b/chrome/browser/ui/views/collected_cookies_views.cc
index 29df017aa0190163b30045adbbcddc3d18200497..a94c2b737cd0609b860ca4e0717983b1a28876ab 100644
--- a/chrome/browser/ui/views/collected_cookies_views.cc
+++ b/chrome/browser/ui/views/collected_cookies_views.cc
@@ -71,14 +71,6 @@ const SkColor kCookiesBorderColor = SkColorSetRGB(0xC8, 0xC8, 0xC8);
const int kTabbedPaneTopPadding = 14;
const int kCookieInfoBottomPadding = 4;
-LayoutDelegate::Metric GetTreeviewToButtonsMetric() {
- // Hack: in the Harmony specs, the buttons under the treeview are "unrelated"
- // to it (which looks better), but in the existing dialog they were related.
- return LayoutDelegate::Get()->IsHarmonyMode()
- ? LayoutDelegate::Metric::UNRELATED_CONTROL_VERTICAL_SPACING
- : LayoutDelegate::Metric::RELATED_CONTROL_VERTICAL_SPACING;
-}
-
} // namespace
// A custom view that conditionally displays an infobar.
@@ -409,7 +401,8 @@ views::View* CollectedCookiesViews::CreateAllowedPane() {
GridLayout::FILL, GridLayout::FILL, kTreeViewWidth,
kTreeViewHeight);
layout->AddPaddingRow(
- 0, LayoutDelegate::Get()->GetMetric(GetTreeviewToButtonsMetric()));
+ 0, LayoutDelegate::Get()->GetMetric(
+ LayoutDelegate::Metric::UNRELATED_CONTROL_VERTICAL_SPACING));
layout->StartRow(0, three_columns_layout_id);
layout->AddView(block_allowed_button_);
@@ -485,7 +478,8 @@ views::View* CollectedCookiesViews::CreateBlockedPane() {
CreateScrollView(blocked_cookies_tree_), 1, 1,
GridLayout::FILL, GridLayout::FILL, kTreeViewWidth, kTreeViewHeight);
layout->AddPaddingRow(
- 0, LayoutDelegate::Get()->GetMetric(GetTreeviewToButtonsMetric()));
+ 0, LayoutDelegate::Get()->GetMetric(
+ LayoutDelegate::Metric::UNRELATED_CONTROL_VERTICAL_SPACING));
layout->StartRow(0, three_columns_layout_id);
layout->AddView(allow_blocked_button_);
« no previous file with comments | « no previous file | chrome/browser/ui/views/device_chooser_content_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698