Broke out layout metric information from ViewsDelegate to LayoutProvider

chrome/browser/ui/views/arc_app_dialog_view.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/app_list/arc/arc_app_dialog.h"
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
 #include "chrome/browser/ui/app_list/arc/arc_app_icon_loader.h"
 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
 #include "chrome/browser/ui/app_list/arc/arc_app_utils.h"
 #include "chrome/browser/ui/native_window_tracker.h"
-#include "chrome/browser/ui/views/harmony/layout_delegate.h"
+#include "chrome/browser/ui/views/harmony/chrome_views_layout_delegate.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/constrained_window/constrained_window_views.h"
 #include "components/strings/grit/components_strings.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/view.h"
@@ skipping 103 matching lines @@
       app_id_(app_id),
       window_title_(window_title),
       confirm_button_text_(confirm_button_text),
       cancel_button_text_(cancel_button_text),
       confirm_callback_(confirm_callback) {
   DCHECK(controller);
   parent_ = controller_->GetAppListWindow();
   if (parent_)
     parent_window_tracker_ = NativeWindowTracker::Create(parent_);
 
+  // TODO: Harmony - review the use of constants below

[Peter Kasting, 2017/03/30 00:49:04]

Nit: I wouldn't add this TODO for now, as we haven't even figured out if this
class is on the list of things we'll be touching.

[kylix_rd, 2017/03/30 19:34:07]

Done.

   SetLayoutManager(new views::BoxLayout(
       views::BoxLayout::kHorizontal, views::kButtonHEdgeMarginNew,
-      LayoutDelegate::Get()->GetMetric(
-          LayoutDelegate::Metric::PANEL_CONTENT_MARGIN),
+      ChromeViewsLayoutDelegate::Get()->GetDistanceMetric(
+          ChromeDistanceMetric::PANEL_CONTENT_MARGIN),
       views::kRelatedControlHorizontalSpacing));
 
   icon_view_ = new FixedBoundarySizeImageView();
   AddChildView(icon_view_);
 
   views::View* text_container = new views::View();
   views::BoxLayout* text_container_layout =
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0);
   text_container_layout->set_main_axis_alignment(
       views::BoxLayout::MAIN_AXIS_ALIGNMENT_CENTER);
@@ skipping 136 matching lines @@
 
 bool CloseAppDialogViewAndConfirmForTest(bool confirm) {
   if (!g_current_arc_app_dialog_view)
     return false;
 
   g_current_arc_app_dialog_view->ConfirmOrCancelForTest(confirm);
   return true;
 }
 
 }  // namespace arc