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

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

Issue 2867253003: Description: Views/Harmony Remove references to ui/views/layout/layout_constants.h (Closed)
Patch Set: Address comments Created 3 years, 7 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
Index: chrome/browser/ui/views/certificate_selector.cc
diff --git a/chrome/browser/ui/views/certificate_selector.cc b/chrome/browser/ui/views/certificate_selector.cc
index c54873795e6ab865ea9da45662e7264ea89813d0..4ae2049f9d15d6252ed46d6ea1a65c373db4bad8 100644
--- a/chrome/browser/ui/views/certificate_selector.cc
+++ b/chrome/browser/ui/views/certificate_selector.cc
@@ -15,6 +15,7 @@
#include "base/strings/utf_string_conversions.h"
#include "build/build_config.h"
#include "chrome/browser/certificate_viewer.h"
+#include "chrome/browser/ui/views/harmony/chrome_layout_provider.h"
#include "chrome/grit/generated_resources.h"
#include "components/constrained_window/constrained_window_views.h"
#include "components/guest_view/browser/guest_view_base.h"
@@ -27,7 +28,6 @@
#include "ui/views/controls/button/md_text_button.h"
#include "ui/views/controls/table/table_view.h"
#include "ui/views/layout/grid_layout.h"
-#include "ui/views/layout/layout_constants.h"
#include "ui/views/widget/widget.h"
#include "ui/views/window/dialog_client_view.h"
@@ -207,7 +207,11 @@ void CertificateSelector::InitWithText(
layout->StartRow(0, kColumnSetId);
layout->AddView(text_label.release());
- layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
+ ChromeLayoutProvider* provider = ChromeLayoutProvider::Get();
+ const int vertical_spacing = provider->GetDistanceMetric(
+ views::DISTANCE_RELATED_CONTROL_VERTICAL);
+
+ layout->AddPaddingRow(0, vertical_spacing);
std::vector<ui::TableColumn> columns;
columns.push_back(ui::TableColumn(IDS_CERT_SELECTOR_SUBJECT_COLUMN,
@@ -228,7 +232,7 @@ void CertificateSelector::InitWithText(
views::GridLayout::FILL, views::GridLayout::FILL,
kTableViewWidth, kTableViewHeight);
- layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, vertical_spacing);
}
ui::TableModel* CertificateSelector::table_model_for_testing() const {
« no previous file with comments | « chrome/browser/ui/views/arc_app_dialog_view.cc ('k') | chrome/browser/ui/views/conflicting_module_view_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698