Chromium Code Reviews| Index: chrome/browser/ui/views/website_settings/permissions_bubble_view.cc |
| diff --git a/chrome/browser/ui/views/website_settings/permissions_bubble_view.cc b/chrome/browser/ui/views/website_settings/permissions_bubble_view.cc |
| index c34f56851f54d1b0f8a3167de6945d9aa91625fb..e3e6a0edf0f98e907c8180d4773ed900d9e222b7 100644 |
| --- a/chrome/browser/ui/views/website_settings/permissions_bubble_view.cc |
| +++ b/chrome/browser/ui/views/website_settings/permissions_bubble_view.cc |
| @@ -68,7 +68,7 @@ PermissionsBubbleDelegateView::PermissionsBubbleDelegateView( |
| set_close_on_deactivate(false); |
| SetLayoutManager(new views::BoxLayout( |
| - views::BoxLayout::kVertical, 30, 0, 20)); |
| + views::BoxLayout::kVertical, 12, 0, 10)); |
|
markusheintz_
2014/02/12 16:52:48
I actually think it would be better to introduce c
Greg Billock
2014/02/13 21:43:39
Yeah, I'll do that. Unfortunately the numbers don'
|
| ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance(); |
| @@ -106,7 +106,7 @@ PermissionsBubbleDelegateView::PermissionsBubbleDelegateView( |
| columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::FILL, |
| 0, views::GridLayout::USE_PREF, 0, 0); |
| if (!customization_mode) { |
| - columns->AddPaddingColumn(0, 10); // padding between buttons |
| + columns->AddPaddingColumn(0, 6); // padding between buttons |
| columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::FILL, |
| 0, views::GridLayout::USE_PREF, 0, 0); |
| } |
| @@ -170,7 +170,7 @@ PermissionsBubbleDelegateView::PermissionsBubbleDelegateView( |
| button_layout->AddView(allow_button); |
| allow_ = allow_button; |
| - button_layout->AddPaddingRow(0, 30); |
| + button_layout->AddPaddingRow(0, 12); |
| } |
| PermissionsBubbleDelegateView::~PermissionsBubbleDelegateView() { |