views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/translate/translate_bubble_view.cc

Index: chrome/browser/ui/views/translate/translate_bubble_view.cc
diff --git a/chrome/browser/ui/views/translate/translate_bubble_view.cc b/chrome/browser/ui/views/translate/translate_bubble_view.cc
index 140c1181340146d832975d29ab6638d2af3f8288..05c58ae42c1c064e7a68f43ce249b2ba6f7ad605 100644
--- a/chrome/browser/ui/views/translate/translate_bubble_view.cc
+++ b/chrome/browser/ui/views/translate/translate_bubble_view.cc
@@ -241,7 +241,7 @@ bool TranslateBubbleView::AcceleratorPressed(
   return BubbleDialogDelegateView::AcceleratorPressed(accelerator);
 }
 
-gfx::Size TranslateBubbleView::GetPreferredSize() const {
+gfx::Size TranslateBubbleView::GetUnsnappedPreferredSize() const {
   int width = 0;

[Peter Kasting, 2017/03/30 00:35:36]

Isn't this the case you said could just be eliminated since it should be handled
by the layout manager?

   for (int i = 0; i < child_count(); i++) {
     const views::View* child = child_at(i);