views: add Harmony dialog width support

ui/views/window/dialog_delegate.cc

 // Copyright (c) 2012 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 "ui/views/window/dialog_delegate.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "build/build_config.h"
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/l10n/l10n_util.h"
+#include "ui/base/material_design/material_design_controller.h"
 #include "ui/gfx/color_palette.h"
 #include "ui/strings/grit/ui_strings.h"
 #include "ui/views/bubble/bubble_border.h"
 #include "ui/views/bubble/bubble_frame_view.h"
 #include "ui/views/controls/button/label_button.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/style/platform_style.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_observer.h"
 #include "ui/views/window/dialog_client_view.h"
@@ skipping 235 matching lines @@
   state->name = GetWindowTitle();
   state->role = ui::AX_ROLE_DIALOG;
 }
 
 void DialogDelegateView::ViewHierarchyChanged(
     const ViewHierarchyChangedDetails& details) {
   if (details.is_add && details.child == this && GetWidget())
     NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
 }
 
+gfx::Size DialogDelegateView::GetPreferredSize() const {
+  if (!ui::MaterialDesignController::IsSecondaryUiMaterial() ||
+      dialog_width() == DIALOG_WIDTH_UNSPECIFIED) {
+    return View::GetPreferredSize();
+  }
+  int width_px = dialog_width() * 32;

[Evan Stade, 2016/10/03 19:04:19]

actually dp, not px. I'd suggest leaving the unit off as  GetPreferredSize
always expects the same thing (dp).

[Elly Fong-Jones, 2016/10/05 13:20:31]

Done.

+  return gfx::Size(width_px, GetHeightForWidth(width_px));
+}
+
 }  // namespace views