Chromium Code Reviews| Index: chrome/browser/ui/views/ime/ime_warning_bubble_view.cc |
| diff --git a/chrome/browser/ui/views/ime/ime_warning_bubble_view.cc b/chrome/browser/ui/views/ime/ime_warning_bubble_view.cc |
| index 24c019ce01ce0e4e4e800b623fcb45242b959774..4233f2684b18c0c06121867e248d5f61fab6dadc 100644 |
| --- a/chrome/browser/ui/views/ime/ime_warning_bubble_view.cc |
| +++ b/chrome/browser/ui/views/ime/ime_warning_bubble_view.cc |
| @@ -12,6 +12,7 @@ |
| #include "chrome/browser/ui/browser_dialogs.h" |
| #include "chrome/browser/ui/browser_list.h" |
| #include "chrome/browser/ui/views/frame/browser_view.h" |
| +#include "chrome/browser/ui/views/harmony/chrome_layout_provider.h" |
| #include "chrome/browser/ui/views/harmony/chrome_typography.h" |
| #include "chrome/browser/ui/views/toolbar/app_menu_button.h" |
| #include "chrome/browser/ui/views/toolbar/browser_actions_container.h" |
| @@ -21,7 +22,6 @@ |
| #include "ui/base/l10n/l10n_util.h" |
| #include "ui/base/resource/resource_bundle.h" |
| #include "ui/views/controls/button/checkbox.h" |
| -#include "ui/views/layout/layout_constants.h" |
| using extensions::Extension; |
| @@ -174,20 +174,23 @@ void ImeWarningBubbleView::InitLayout() { |
| main_cs->AddColumn(views::GridLayout::LEADING, views::GridLayout::LEADING, 0, |
| views::GridLayout::FIXED, kColumnWidth, 0); |
| + ChromeLayoutProvider* provider = ChromeLayoutProvider::Get(); |
| + const int vertical_spacing = |
| + provider->GetDistanceMetric(views::DISTANCE_RELATED_CONTROL_VERTICAL); |
| layout->StartRow(0, cs_id); |
| base::string16 extension_name = base::UTF8ToUTF16(extension_->name()); |
| base::i18n::AdjustStringForLocaleDirection(&extension_name); |
| views::Label* warning = CreateExtensionNameLabel(l10n_util::GetStringFUTF16( |
| IDS_IME_API_ACTIVATED_WARNING, extension_name)); |
| layout->AddView(warning); |
| - layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing); |
| + layout->AddPaddingRow(0, vertical_spacing); |
| // The seconde row which shows the check box. |
| layout->StartRow(0, cs_id); |
| never_show_checkbox_ = |
| new views::Checkbox(l10n_util::GetStringUTF16(IDS_IME_API_NEVER_SHOW)); |
| layout->AddView(never_show_checkbox_); |
| - layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing); |
| + layout->AddPaddingRow(0, vertical_spacing); |
|
Peter Kasting
2017/05/26 00:50:24
Why does this padding row get added at all? It se
ananta
2017/05/26 03:23:24
Done.
|
| } |
| bool ImeWarningBubbleView::IsToolbarAnimating() { |