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

Unified Diff: chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc

Issue 2907933002: Views/Harmony Remove references to ui/views/layout/layout_constants.h (Closed)
Patch Set: Remove unused dialog_insets 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/apps/app_info_dialog/app_info_footer_panel.cc
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc b/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
index 825c43d00d055fb426d7781b6bb39dcd41df61c0..c834d03af91b4a05901738307224373406e9d49c 100644
--- a/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
@@ -6,6 +6,7 @@
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser_dialogs.h"
+#include "chrome/browser/ui/views/harmony/chrome_layout_provider.h"
#include "chrome/common/extensions/extension_constants.h"
#include "chrome/grit/generated_resources.h"
#include "extensions/browser/extension_system.h"
@@ -16,7 +17,6 @@
#include "ui/events/event.h"
#include "ui/views/controls/button/md_text_button.h"
#include "ui/views/layout/box_layout.h"
-#include "ui/views/layout/layout_constants.h"
#include "ui/views/view.h"
#include "ui/views/widget/widget.h"
@@ -38,10 +38,15 @@ AppInfoFooterPanel::AppInfoFooterPanel(gfx::NativeWindow parent_window,
weak_ptr_factory_(this) {
CreateButtons();
- SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal,
- views::kButtonHEdgeMargin,
- views::kButtonVEdgeMargin,
- views::kRelatedButtonHSpacing));
+ ChromeLayoutProvider* provider = ChromeLayoutProvider::Get();
+
+ SetLayoutManager(new views::BoxLayout(
+ views::BoxLayout::kHorizontal,
+ provider->GetDistanceMetric(
+ views::DISTANCE_DIALOG_CONTENTS_HORIZONTAL_MARGIN),
+ provider->GetDistanceMetric(
+ views::DISTANCE_DIALOG_CONTENTS_VERTICAL_MARGIN),
+ provider->GetDistanceMetric(views::DISTANCE_RELATED_BUTTON_HORIZONTAL)));
LayoutButtons();
}

Powered by Google App Engine
This is Rietveld 408576698