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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/views/try_chrome_dialog_view.cc

Issue 2816293002: Description: Replace layout constants in chrome/browser/extensions and chrome/browser/first_run (Closed)
Patch Set: Fix dumb error Created 3 years, 8 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
« chrome/browser/ui/views/extensions/request_file_system_dialog_view.cc ('K') | « chrome/browser/ui/views/try_chrome_dialog_view.h ('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/try_chrome_dialog_view.cc
diff --git a/chrome/browser/first_run/try_chrome_dialog_view.cc b/chrome/browser/ui/views/try_chrome_dialog_view.cc
similarity index 88%
rename from chrome/browser/first_run/try_chrome_dialog_view.cc
rename to chrome/browser/ui/views/try_chrome_dialog_view.cc
index 4696d13428be980b5e332e55fbadf00f9c344dd4..67406168a2654281d002df69713d9ba352b0ff94 100644
--- a/chrome/browser/first_run/try_chrome_dialog_view.cc
+++ b/chrome/browser/ui/views/try_chrome_dialog_view.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/first_run/try_chrome_dialog_view.h"
+#include "chrome/browser/ui/views/try_chrome_dialog_view.h"
#include <shellapi.h>
@@ -12,6 +12,7 @@
#include "base/run_loop.h"
#include "base/strings/string16.h"
#include "chrome/browser/process_singleton.h"
+#include "chrome/browser/ui/views/harmony/chrome_layout_provider.h"
#include "chrome/grit/chromium_strings.h"
#include "chrome/grit/generated_resources.h"
#include "chrome/grit/theme_resources.h"
@@ -33,7 +34,6 @@
#include "ui/views/controls/link.h"
#include "ui/views/controls/separator.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/widget/widget.h"
@@ -74,11 +74,9 @@ TryChromeDialogView::TryChromeDialogView(size_t flavor)
kill_chrome_(NULL),
dont_try_chrome_(NULL),
make_default_(NULL),
- result_(COUNT) {
-}
+ result_(COUNT) {}
-TryChromeDialogView::~TryChromeDialogView() {
-}
+TryChromeDialogView::~TryChromeDialogView() {}
TryChromeDialogView::Result TryChromeDialogView::ShowModal(
const ActiveModalDialogListener& listener) {
@@ -103,46 +101,54 @@ TryChromeDialogView::Result TryChromeDialogView::ShowModal(
views::GridLayout* layout = views::GridLayout::CreatePanel(root_view);
views::ColumnSet* columns;
+ ChromeLayoutProvider* provider = ChromeLayoutProvider::Get();
+ const int label_spacing =
+ provider->GetDistanceMetric(DISTANCE_RELATED_LABEL_HORIZONTAL);
+ const int unrelated_space_horiz =
+ provider->GetDistanceMetric(DISTANCE_UNRELATED_CONTROL_HORIZONTAL);
+ const int unrelated_space_vert =
+ provider->GetDistanceMetric(DISTANCE_UNRELATED_CONTROL_VERTICAL);
+ const int button_spacing_horiz =
+ provider->GetDistanceMetric(views::DISTANCE_RELATED_BUTTON_HORIZONTAL);
+
// First row: [icon][pad][text][pad][button].
columns = layout->AddColumnSet(0);
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::LEADING, 0,
views::GridLayout::FIXED, icon_size.width(),
icon_size.height());
- columns->AddPaddingColumn(0, views::kRelatedControlHorizontalSpacing);
+ columns->AddPaddingColumn(0, label_spacing);
columns->AddColumn(views::GridLayout::FILL, views::GridLayout::FILL, 1,
views::GridLayout::USE_PREF, 0, 0);
- columns->AddPaddingColumn(0, views::kUnrelatedControlHorizontalSpacing);
+ columns->AddPaddingColumn(0, unrelated_space_horiz);
columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::FILL, 1,
views::GridLayout::USE_PREF, 0, 0);
+ int icon_padding = icon_size.width() + label_spacing;
// Optional second row: [pad][pad][radio 1].
Peter Kasting 2017/04/26 18:26:25 Nit: This and comments below that say [pad][pad] s
ananta 2017/04/26 19:42:21 Done.
columns = layout->AddColumnSet(1);
- columns->AddPaddingColumn(0, icon_size.width());
- columns->AddPaddingColumn(0, views::kRelatedControlHorizontalSpacing);
+ columns->AddPaddingColumn(0, icon_padding);
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::FILL, 1,
views::GridLayout::USE_PREF, 0, 0);
// Third row: [pad][pad][radio 2].
columns = layout->AddColumnSet(2);
- columns->AddPaddingColumn(0, icon_size.width());
- columns->AddPaddingColumn(0, views::kRelatedControlHorizontalSpacing);
+ columns->AddPaddingColumn(0, icon_padding);
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::FILL, 1,
views::GridLayout::USE_PREF, 0, 0);
// Fourth row: [pad][pad][button][pad][button].
columns = layout->AddColumnSet(3);
- columns->AddPaddingColumn(0, icon_size.width());
+ columns->AddPaddingColumn(0, icon_padding);
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::FILL, 0,
views::GridLayout::USE_PREF, 0, 0);
- columns->AddPaddingColumn(0, views::LayoutProvider::Get()->GetDistanceMetric(
- views::DISTANCE_RELATED_BUTTON_HORIZONTAL));
+ columns->AddPaddingColumn(0, button_spacing_horiz);
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::FILL, 0,
views::GridLayout::USE_PREF, 0, 0);
// Fifth row: [pad][pad][link].
columns = layout->AddColumnSet(4);
columns->AddPaddingColumn(0, icon_size.width());
- columns->AddPaddingColumn(0, views::kRelatedControlHorizontalSpacing);
+ columns->AddPaddingColumn(0, label_spacing);
Peter Kasting 2017/04/26 18:26:25 Nit: This should be one padding column of size |ic
ananta 2017/04/26 19:42:21 Thanks done.
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::FILL, 1,
views::GridLayout::USE_PREF, 0, 0);
@@ -160,7 +166,7 @@ TryChromeDialogView::Result TryChromeDialogView::ShowModal(
columns = layout->AddColumnSet(7);
columns->AddColumn(views::GridLayout::LEADING, views::GridLayout::FILL, 0,
views::GridLayout::USE_PREF, 0, 0);
- columns->AddPaddingColumn(0, views::kUnrelatedControlHorizontalSpacing);
+ columns->AddPaddingColumn(0, unrelated_space_horiz);
columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::FILL, 1,
views::GridLayout::USE_PREF, 0, 0);
@@ -230,13 +236,13 @@ TryChromeDialogView::Result TryChromeDialogView::ShowModal(
views::Separator* separator = NULL;
if (experiment.flags & installer::kToastUiMakeDefault) {
- // In this flavor we have some veritical space, then a separator line
+ // In this flavor we have some vertical space, then a separator line
// and the 'make default' checkbox and the OK button on the same row.
- layout->AddPaddingRow(0, views::kUnrelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, unrelated_space_vert);
layout->StartRow(0, 6);
separator = new views::Separator();
layout->AddView(separator);
- layout->AddPaddingRow(0, views::kUnrelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, unrelated_space_vert);
layout->StartRow(0, 7);
make_default_ = new views::Checkbox(
@@ -262,8 +268,8 @@ TryChromeDialogView::Result TryChromeDialogView::ShowModal(
if (experiment.flags & installer::kToastUiWhyLink) {
layout->StartRowWithPadding(0, 4, 0, 10);
- views::Link* link = new views::Link(
- l10n_util::GetStringUTF16(IDS_TRY_TOAST_WHY));
+ views::Link* link =
+ new views::Link(l10n_util::GetStringUTF16(IDS_TRY_TOAST_WHY));
link->set_listener(this);
layout->AddView(link);
}
@@ -315,10 +321,9 @@ gfx::Rect TryChromeDialogView::ComputeWindowPosition(gfx::Size size,
void TryChromeDialogView::SetToastRegion(HWND window, int w, int h) {
static const POINT polygon[] = {
- {0, 4}, {1, 2}, {2, 1}, {4, 0}, // Left side.
- {w-4, 0}, {w-2, 1}, {w-1, 2}, {w, 4}, // Right side.
- {w, h}, {0, h}
- };
+ {0, 4}, {1, 2}, {2, 1}, {4, 0}, // Left side.
+ {w - 4, 0}, {w - 2, 1}, {w - 1, 2}, {w, 4}, // Right side.
+ {w, h}, {0, h}};
HRGN region = ::CreatePolygonRgn(polygon, arraysize(polygon), WINDING);
::SetWindowRgn(window, region, FALSE);
}
@@ -357,7 +362,7 @@ void TryChromeDialogView::ButtonPressed(views::Button* sender,
if (make_default_) {
if ((result_ == TRY_CHROME) && make_default_->checked())
- result_ = TRY_CHROME_AS_DEFAULT;
+ result_ = TRY_CHROME_AS_DEFAULT;
}
popup_->Close();
« chrome/browser/ui/views/extensions/request_file_system_dialog_view.cc ('K') | « chrome/browser/ui/views/try_chrome_dialog_view.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698