Commit 8b6a69f0 authored by patricialor's avatar patricialor Committed by Commit bot

Views/Harmony: Remove references to layout constants in c/b/ui/views/sync.

Remove unnecessary includes for ui/views/layout/layout_constants.h and do some
refactoring to remove references to layout constants in
ProfileSigninConfirmationDialogViews under chrome/browser/ui/views/sync/*.

See
https://drive.google.com/file/d/0BzEa5HU1aAqBOXBDazdRT245Wk0/view?usp=sharing
for before/after screenshots of ProfileSigninConfirmationDialogViews.

BUG=691897

Review-Url: https://codereview.chromium.org/2799163002
Cr-Commit-Position: refs/heads/master@{#462771}
parent 84913869
...@@ -13,7 +13,6 @@ ...@@ -13,7 +13,6 @@
#include "ui/gfx/font.h" #include "ui/gfx/font.h"
#include "ui/views/controls/styled_label.h" #include "ui/views/controls/styled_label.h"
#include "ui/views/layout/fill_layout.h" #include "ui/views/layout/fill_layout.h"
#include "ui/views/layout/layout_constants.h"
namespace { namespace {
......
...@@ -25,7 +25,6 @@ ...@@ -25,7 +25,6 @@
#include "ui/views/controls/label.h" #include "ui/views/controls/label.h"
#include "ui/views/controls/link.h" #include "ui/views/controls/link.h"
#include "ui/views/layout/grid_layout.h" #include "ui/views/layout/grid_layout.h"
#include "ui/views/layout/layout_constants.h"
#include "ui/views/widget/widget.h" #include "ui/views/widget/widget.h"
namespace { namespace {
......
...@@ -13,7 +13,6 @@ ...@@ -13,7 +13,6 @@
#include "chrome/browser/ui/browser_navigator.h" #include "chrome/browser/ui/browser_navigator.h"
#include "chrome/browser/ui/browser_navigator_params.h" #include "chrome/browser/ui/browser_navigator_params.h"
#include "chrome/browser/ui/browser_window.h" #include "chrome/browser/ui/browser_window.h"
#include "chrome/browser/ui/views/harmony/layout_delegate.h"
#include "chrome/grit/chromium_strings.h" #include "chrome/grit/chromium_strings.h"
#include "chrome/grit/generated_resources.h" #include "chrome/grit/generated_resources.h"
#include "components/constrained_window/constrained_window_views.h" #include "components/constrained_window/constrained_window_views.h"
...@@ -33,7 +32,7 @@ ...@@ -33,7 +32,7 @@
#include "ui/views/controls/styled_label.h" #include "ui/views/controls/styled_label.h"
#include "ui/views/layout/box_layout.h" #include "ui/views/layout/box_layout.h"
#include "ui/views/layout/grid_layout.h" #include "ui/views/layout/grid_layout.h"
#include "ui/views/layout/layout_constants.h" #include "ui/views/views_delegate.h"
#include "ui/views/widget/widget.h" #include "ui/views/widget/widget.h"
#include "ui/views/window/dialog_client_view.h" #include "ui/views/window/dialog_client_view.h"
...@@ -194,18 +193,22 @@ void ProfileSigninConfirmationDialogViews::ViewHierarchyChanged( ...@@ -194,18 +193,22 @@ void ProfileSigninConfirmationDialogViews::ViewHierarchyChanged(
views::StyledLabel::RangeStyleInfo::CreateForLink()); views::StyledLabel::RangeStyleInfo::CreateForLink());
// Layout the components. // Layout the components.
const int panel_margin = LayoutDelegate::Get()->GetMetric( const gfx::Insets panel_insets =
LayoutDelegate::Metric::PANEL_CONTENT_MARGIN); views::ViewsDelegate::GetInstance()->GetInsetsMetric(
views::InsetsMetric::PANEL);
// The prompt bar needs to go to the edge of the dialog, so ignore insets for
// the outer layout.
views::GridLayout* dialog_layout = new views::GridLayout(this); views::GridLayout* dialog_layout = new views::GridLayout(this);
dialog_layout->SetInsets(panel_margin, 0, 0, 0); dialog_layout->SetInsets(panel_insets.top(), 0, panel_insets.bottom(), 0);
SetLayoutManager(dialog_layout); SetLayoutManager(dialog_layout);
// Use GridLayout inside the prompt bar because StyledLabel requires it. // Use GridLayout inside the prompt bar because StyledLabel requires it.
views::GridLayout* prompt_layout = views::GridLayout::CreatePanel(prompt_bar); views::GridLayout* prompt_layout = views::GridLayout::CreatePanel(prompt_bar);
prompt_layout->AddColumnSet(0)->AddColumn( constexpr int kPromptBarColumnSetId = 0;
views::GridLayout::FILL, views::GridLayout::CENTER, 100, prompt_layout->AddColumnSet(kPromptBarColumnSetId)
views::GridLayout::USE_PREF, 0, 0); ->AddColumn(views::GridLayout::FILL, views::GridLayout::CENTER, 100,
prompt_layout->StartRow(0, 0); views::GridLayout::USE_PREF, 0, 0);
prompt_layout->StartRow(0, kPromptBarColumnSetId);
prompt_layout->AddView(prompt_label); prompt_layout->AddView(prompt_label);
// Use a column set with no padding. // Use a column set with no padding.
dialog_layout->AddColumnSet(0)->AddColumn( dialog_layout->AddColumnSet(0)->AddColumn(
...@@ -217,14 +220,16 @@ void ProfileSigninConfirmationDialogViews::ViewHierarchyChanged( ...@@ -217,14 +220,16 @@ void ProfileSigninConfirmationDialogViews::ViewHierarchyChanged(
views::GridLayout::FILL, views::GridLayout::FILL, 0, 0); views::GridLayout::FILL, views::GridLayout::FILL, 0, 0);
// Use a new column set for the explanation label so we can add padding. // Use a new column set for the explanation label so we can add padding.
dialog_layout->AddPaddingRow(0.0, panel_margin); dialog_layout->AddPaddingRow(0.0, panel_insets.top());
views::ColumnSet* explanation_columns = dialog_layout->AddColumnSet(1); constexpr int kExplanationColumnSetId = 1;
explanation_columns->AddPaddingColumn(0.0, views::kButtonHEdgeMarginNew); views::ColumnSet* explanation_columns =
dialog_layout->AddColumnSet(kExplanationColumnSetId);
explanation_columns->AddPaddingColumn(0.0, panel_insets.left());
explanation_columns->AddColumn( explanation_columns->AddColumn(
views::GridLayout::FILL, views::GridLayout::FILL, 100, views::GridLayout::FILL, views::GridLayout::FILL, 100,
views::GridLayout::USE_PREF, 0, 0); views::GridLayout::USE_PREF, 0, 0);
explanation_columns->AddPaddingColumn(0.0, views::kButtonHEdgeMarginNew); explanation_columns->AddPaddingColumn(0.0, panel_insets.right());
dialog_layout->StartRow(0, 1); dialog_layout->StartRow(0, kExplanationColumnSetId);
const int kPreferredWidth = 440; const int kPreferredWidth = 440;
dialog_layout->AddView(explanation_label, 1, 1, views::GridLayout::FILL, dialog_layout->AddView(explanation_label, 1, 1, views::GridLayout::FILL,
views::GridLayout::FILL, kPreferredWidth, views::GridLayout::FILL, kPreferredWidth,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment