Chromium Code Reviews| Index: chrome/browser/chromeos/profiles/multiprofiles_session_aborted_dialog.cc |
| diff --git a/chrome/browser/chromeos/profiles/multiprofiles_session_aborted_dialog.cc b/chrome/browser/chromeos/profiles/multiprofiles_session_aborted_dialog.cc |
| index 24c926314434ae60b20bb165e916992f5e07f79e..e2e33a73dc641504dffab4639775af5ddfe5b3b7 100644 |
| --- a/chrome/browser/chromeos/profiles/multiprofiles_session_aborted_dialog.cc |
| +++ b/chrome/browser/chromeos/profiles/multiprofiles_session_aborted_dialog.cc |
| @@ -28,8 +28,6 @@ const int kDefaultWidth = 600; |
| const int kDefaultHeight = 250; |
| const int kPaddingToMessage = 20; |
| -const int kInset = 40; |
| -const int kTopInset = 10; |
| //////////////////////////////////////////////////////////////////////////////// |
| // Dialog for an aborted multi-profile session due to a user policy change . |
| @@ -113,11 +111,11 @@ gfx::Size MultiprofilesSessionAbortedView::GetPreferredSize() const { |
| void MultiprofilesSessionAbortedView::InitDialog( |
| const std::string& user_email) { |
| - const gfx::Insets kDialogInsets(kTopInset, kInset, kInset, kInset); |
| - |
| + constexpr int kTopInset = 10; |
| + constexpr int kInset = 40; |
|
sky
2017/05/05 14:36:39
optional: kInset is rather generic, maybe kOtherIn
Bret
2017/05/06 00:36:43
Done.
|
| // Create the views and layout manager and set them up. |
| views::GridLayout* grid_layout = views::GridLayout::CreatePanel(this); |
|
sky
2017/05/05 14:36:39
As the next line sets the border, how about not us
Bret
2017/05/06 00:36:43
Done. Though like in the previous patch, my plan i
|
| - grid_layout->SetInsets(kDialogInsets); |
| + SetBorder(CreateEmptyBorder(kTopInset, kInset, kInset, kInset)); |
| views::ColumnSet* column_set = grid_layout->AddColumnSet(0); |
| column_set->AddColumn(views::GridLayout::FILL, views::GridLayout::FILL, 1, |