views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/extensions/extension_popup.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 "chrome/browser/ui/views/extensions/extension_popup.h"
 
 #include "base/bind.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/devtools/devtools_window.h"
 #include "chrome/browser/extensions/extension_view_host.h"
@@ skipping 126 matching lines @@
     return;
   // Widget::Close posts a task, which should give the devtools window a
   // chance to finish detaching from the inspected RenderViewHost.
   GetWidget()->Close();
 }
 
 void ExtensionPopup::OnExtensionSizeChanged(ExtensionViewViews* view) {
   SizeToContents();
 }
 
-gfx::Size ExtensionPopup::GetPreferredSize() const {
+gfx::Size ExtensionPopup::GetUnsnappedPreferredSize() const {
   // Constrain the size to popup min/max.
   gfx::Size sz = views::View::GetPreferredSize();
   sz.set_width(std::max(kMinWidth, std::min(kMaxWidth, sz.width())));
   sz.set_height(std::max(kMinHeight, std::min(kMaxHeight, sz.height())));
   return sz;
 }
 
 void ExtensionPopup::ViewHierarchyChanged(
   const ViewHierarchyChangedDetails& details) {
   // TODO(msw): Find any remaining crashes related to http://crbug.com/327776
@@ skipping 39 matching lines @@
   GetWidget()->Show();
 
   // Focus on the host contents when the bubble is first shown.
   host()->host_contents()->Focus();
 
   if (inspect_with_devtools_) {
     DevToolsWindow::OpenDevToolsWindow(
         host()->host_contents(), DevToolsToggleAction::ShowConsolePanel());
   }
 }