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

Unified Diff: chrome/browser/ui/views/profiles/forced_reauthentication_dialog.cc

Issue 2928253002: Remove references to ui/views/layout/layout_constants.h (Closed)
Patch Set: Fix build errors Created 3 years, 6 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 | « chrome/browser/ui/views/chrome_cleaner_dialog_win.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/profiles/forced_reauthentication_dialog.cc
diff --git a/chrome/browser/ui/views/profiles/forced_reauthentication_dialog.cc b/chrome/browser/ui/views/profiles/forced_reauthentication_dialog.cc
index dcf0c48963f9256bdf536bb7b0eb3ec314b38481..d02b9265849fc78e230351abdbc301786206f4d5 100644
--- a/chrome/browser/ui/views/profiles/forced_reauthentication_dialog.cc
+++ b/chrome/browser/ui/views/profiles/forced_reauthentication_dialog.cc
@@ -18,6 +18,7 @@
#include "chrome/browser/ui/sync/profile_signin_confirmation_helper.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
#include "chrome/browser/ui/views/frame/browser_view.h"
+#include "chrome/browser/ui/views/harmony/chrome_layout_provider.h"
#include "chrome/grit/chromium_strings.h"
#include "chrome/grit/generated_resources.h"
#include "components/constrained_window/constrained_window_views.h"
@@ -27,8 +28,6 @@
#include "ui/views/border.h"
#include "ui/views/controls/styled_label.h"
#include "ui/views/layout/grid_layout.h"
-#include "ui/views/layout/layout_constants.h"
-#include "ui/views/layout/layout_provider.h"
#include "ui/views/view.h"
#include "ui/views/window/dialog_client_view.h"
@@ -76,7 +75,7 @@ class PromptLabel : public views::StyledLabel {
: views::StyledLabel(text, listener) {}
gfx::Insets GetInsets() const override {
- return views::LayoutProvider::Get()->GetInsetsMetric(
+ return ChromeLayoutProvider::Get()->GetInsetsMetric(
views::INSETS_DIALOG_CONTENTS);
}
};
@@ -199,12 +198,12 @@ void ForcedReauthenticationDialog::AddedToWidget() {
views::StyledLabel* explanation_label =
new views::StyledLabel(signin_explanation_text, nullptr);
+ ChromeLayoutProvider* provider = ChromeLayoutProvider::Get();
// Layout the components.
- const gfx::Insets panel_insets =
- views::LayoutProvider::Get()->GetInsetsMetric(
- views::INSETS_DIALOG_CONTENTS);
- SetBorder(views::CreateEmptyBorder(panel_insets.top(), 0,
- panel_insets.bottom(), 0));
+ const gfx::Insets dialog_insets =
+ provider->GetInsetsMetric(views::INSETS_DIALOG_CONTENTS);
+ SetBorder(views::CreateEmptyBorder(dialog_insets.top(), 0,
+ dialog_insets.bottom(), 0));
views::GridLayout* dialog_layout = new views::GridLayout(this);
SetLayoutManager(dialog_layout);
@@ -217,13 +216,14 @@ void ForcedReauthenticationDialog::AddedToWidget() {
views::GridLayout::FILL, 0, 0);
// Use a new column set for the explanation label so we can add padding.
- dialog_layout->AddPaddingRow(0.0, views::kPanelVertMargin);
+ dialog_layout->AddPaddingRow(0.0, dialog_insets.top());
views::ColumnSet* explanation_columns = dialog_layout->AddColumnSet(1);
- explanation_columns->AddPaddingColumn(0.0, views::kButtonHEdgeMarginNew);
+
+ explanation_columns->AddPaddingColumn(0.0, dialog_insets.left());
explanation_columns->AddColumn(views::GridLayout::FILL,
views::GridLayout::FILL, 100,
views::GridLayout::USE_PREF, 0, 0);
- explanation_columns->AddPaddingColumn(0.0, views::kButtonHEdgeMarginNew);
+ explanation_columns->AddPaddingColumn(0.0, dialog_insets.right());
dialog_layout->StartRow(0, 1);
const int kPreferredWidth = 440;
dialog_layout->AddView(explanation_label, 1, 1, views::GridLayout::FILL,
« no previous file with comments | « chrome/browser/ui/views/chrome_cleaner_dialog_win.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698