views: implement dialog width snapping

ui/views/bubble/bubble_frame_view.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/bubble/bubble_frame_view.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "build/build_config.h"
@@ skipping 18 matching lines @@
 #include "ui/views/controls/button/image_button_factory.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/resources/grit/views_resources.h"
 #include "ui/views/views_delegate.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_delegate.h"
 #include "ui/views/window/client_view.h"
+#include "ui/views/window/dialog_delegate.h"
 
 namespace views {
 
 namespace {
 
 // Background color of the footnote view.
 const SkColor kFootnoteBackgroundColor = SkColorSetRGB(245, 245, 245);
 
 // Color of the top border of the footnote.
 const SkColor kFootnoteBorderColor = SkColorSetRGB(229, 229, 229);
@@ skipping 474 matching lines @@
   if (close_->visible())
     title_bar_width += close_->width() + 1;
 
   gfx::Size size(client_size);
   gfx::Insets client_insets = GetInsets();
   size.Enlarge(client_insets.width(), client_insets.height());
   size.SetToMax(gfx::Size(title_bar_width, 0));
 
   if (footnote_container_)
     size.Enlarge(0, footnote_container_->GetHeightForWidth(size.width()));
+  DialogDelegate* as_dialog =

[Peter Kasting, 2017/04/06 06:32:16]

Nit: I'd name this |dialog| or |dialog_delegate|

+      GetWidget()->widget_delegate()->AsDialogDelegate();

[Peter Kasting, 2017/04/06 06:32:16]

Nit: Move this line below the next comment since it's part of that block

+
+  // Only apply width snapping to bubbles that are also dialogs, and let dialogs
+  // opt out of width snapping if they handle it themselves or are otherwise
+  // incompatible with it.
+  if (as_dialog && as_dialog->ShouldSnapFrameWidth()) {
+    size.set_width(
+        ViewsDelegate::GetInstance()->GetSnappedDialogWidth(size.width()));
+  }
 
   return size;
 }
 
 }  // namespace views