Description: Replace layout constants in chrome/browser/extensions and chrome/browser/first_run

chrome/browser/ui/views/extensions/request_file_system_dialog_view.cc

 // Copyright 2015 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/extensions/api/file_system/request_file_system_dialog_view.h"
+#include "chrome/browser/ui/views/extensions/request_file_system_dialog_view.h"
 
 #include <stddef.h>
 
 #include <cstdlib>
 
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_task_runner_handle.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 "content/public/browser/web_contents.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/font.h"
 #include "ui/gfx/range/range.h"
 #include "ui/views/controls/styled_label.h"
-#include "ui/views/layout/box_layout.h"
-#include "ui/views/layout/layout_constants.h"
+#include "ui/views/layout/fill_layout.h"
 
 namespace {
 
 // Maximum width of the dialog in pixels.
 const int kDialogMaxWidth = 320;
 
 }  // namespace
 
 // static
 void RequestFileSystemDialogView::ShowDialog(
     content::WebContents* web_contents,
     const extensions::Extension& extension,
     base::WeakPtr<file_manager::Volume> volume,
     bool writable,
     const base::Callback<void(ui::DialogButton)>& callback) {
   constrained_window::ShowWebModalDialogViews(
       new RequestFileSystemDialogView(extension, volume, writable, callback),
       web_contents);
 }
 
-RequestFileSystemDialogView::~RequestFileSystemDialogView() {
-}
+RequestFileSystemDialogView::~RequestFileSystemDialogView() {}
 
 base::string16 RequestFileSystemDialogView::GetAccessibleWindowTitle() const {
   return l10n_util::GetStringUTF16(
       IDS_FILE_SYSTEM_REQUEST_FILE_SYSTEM_DIALOG_TITLE);
 }
 
 int RequestFileSystemDialogView::GetDefaultDialogButton() const {
   return ui::DIALOG_BUTTON_CANCEL;
 }
 
 base::string16 RequestFileSystemDialogView::GetDialogButtonLabel(
     ui::DialogButton button) const {
   switch (button) {
     case ui::DIALOG_BUTTON_OK:
       return l10n_util::GetStringUTF16(
           IDS_FILE_SYSTEM_REQUEST_FILE_SYSTEM_DIALOG_ALLOW_BUTTON);
     case ui::DIALOG_BUTTON_CANCEL:
       return l10n_util::GetStringUTF16(
           IDS_FILE_SYSTEM_REQUEST_FILE_SYSTEM_DIALOG_DENY_BUTTON);
     default:
       NOTREACHED();
   }
   return base::string16();
 }
 
 ui::ModalType RequestFileSystemDialogView::GetModalType() const {
   return ui::MODAL_TYPE_CHILD;
 }
 
-views::View* RequestFileSystemDialogView::GetContentsView() {
-  return contents_view_;
-}
-
-views::Widget* RequestFileSystemDialogView::GetWidget() {
-  return contents_view_->GetWidget();
-}
-
-const views::Widget* RequestFileSystemDialogView::GetWidget() const {
-  return contents_view_->GetWidget();
-}
-
 bool RequestFileSystemDialogView::Cancel() {
   callback_.Run(ui::DIALOG_BUTTON_CANCEL);
   return true;
 }
 
 bool RequestFileSystemDialogView::Accept() {
   callback_.Run(ui::DIALOG_BUTTON_OK);
   return true;
 }
 
+gfx::Size RequestFileSystemDialogView::GetPreferredSize() const {
+  return gfx::Size(kDialogMaxWidth,
+                   child_at(0)->GetHeightForWidth(kDialogMaxWidth));
+}
+
+gfx::Insets RequestFileSystemDialogView::GetInsets() const {
+  return ChromeLayoutProvider::Get()->GetInsetsMetric(views::INSETS_PANEL);
+}
+
 RequestFileSystemDialogView::RequestFileSystemDialogView(
     const extensions::Extension& extension,
     base::WeakPtr<file_manager::Volume> volume,
     bool writable,
     const base::Callback<void(ui::DialogButton)>& callback)
     : callback_(callback), contents_view_(new views::View) {
   DCHECK(!callback_.is_null());
 
   // If the volume is gone, then cancel the dialog.
   if (!volume.get()) {
@@ skipping 20 matching lines @@
 
   DCHECK_EQ(2u, placeholder_offsets.size());
   label->AddStyleRange(gfx::Range(placeholder_offsets[0],
                                   placeholder_offsets[0] + app_name.length()),
                        bold_style);
   label->AddStyleRange(
       gfx::Range(placeholder_offsets[1],
                  placeholder_offsets[1] + volume_name.length()),
       bold_style);
 
-  views::BoxLayout* const layout = new views::BoxLayout(
-      views::BoxLayout::kHorizontal, views::kButtonHEdgeMarginNew,
-      views::kPanelVertMargin, 0);
-  contents_view_->SetLayoutManager(layout);
+  views::FillLayout* const layout = new views::FillLayout();

[Peter Kasting, 2017/04/26 18:26:25]

Nit: Or just inline into next line.

[ananta, 2017/04/26 19:42:20]

Done.

+  SetLayoutManager(layout);
 
-  label->SizeToFit(kDialogMaxWidth - 2 * views::kButtonHEdgeMarginNew);
-  contents_view_->AddChildView(label);
+  AddChildView(label);
 }