views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/extensions/extension_popup.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_POPUP_H_
 #define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_POPUP_H_
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
@@ skipping 51 matching lines @@
   int GetDialogButtons() const override;
 
   // content::NotificationObserver overrides.
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override;
 
   // ExtensionViewViews::Container overrides.
   void OnExtensionSizeChanged(ExtensionViewViews* view) override;
 
   // views::View overrides.

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

This class doesn't directly subclass View or DialogDelegateView, so really all
those overrides belong with the BubbleDialogDelegateView overrides.

-  gfx::Size GetPreferredSize() const override;
   void ViewHierarchyChanged(
       const ViewHierarchyChangedDetails& details) override;
 
+  // views::DialogDelegateView overrides.
+  gfx::Size GetUnsnappedPreferredSize() const override;
+
   // views::WidgetObserver overrides.
   void OnWidgetActivationChanged(views::Widget* widget, bool active) override;
 
   // TabStripModelObserver overrides.
   void ActiveTabChanged(content::WebContents* old_contents,
                         content::WebContents* new_contents,
                         int index,
                         int reason) override;
 
   // The min/max height of popups.
@@ skipping 34 matching lines @@
   bool inspect_with_devtools_;
 
   content::NotificationRegistrar registrar_;
 
   bool widget_initialized_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionPopup);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_POPUP_H_