views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/profiles/user_manager_view.h

 // Copyright 2014 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_PROFILES_USER_MANAGER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_PROFILES_USER_MANAGER_VIEW_H_
 
 #include <memory>
 
 #include "base/auto_reset.h"
@@ skipping 25 matching lines @@
   void DisplayErrorMessage();
 
  private:
   UserManagerProfileDialogDelegate();
 
   // Before its destruction, tells its parent container to reset its reference
   // to the UserManagerProfileDialogDelegate.
   void OnDialogDestroyed();
 
   // views::DialogDelegate:
-  gfx::Size GetPreferredSize() const override;
+  gfx::Size GetUnsnappedPreferredSize() const override;
   bool CanResize() const override;
   bool CanMaximize() const override;
   bool CanMinimize() const override;
   bool ShouldUseCustomFrame() const override;
   ui::ModalType GetModalType() const override;
   void DeleteDelegate() override;
   base::string16 GetWindowTitle() const override;
   int GetDialogButtons() const override;
   views::View* GetInitiallyFocusedView() override;
 
@@ skipping 54 matching lines @@
   ~UserManagerView() override;
 
   // Resets delegate_ to nullptr when delegate_ is no longer alive.
   void OnDialogDestroyed();
 
   // Creates dialog and initializes UI.
   void Init(Profile* guest_profile, const GURL& url);
 
   // views::View:
   bool AcceleratorPressed(const ui::Accelerator& accelerator) override;
-  gfx::Size GetPreferredSize() const override;
 
   // views::DialogDelegateView:

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

Nit: Combine with above section

   bool CanResize() const override;
   bool CanMaximize() const override;
   bool CanMinimize() const override;
+  gfx::Size GetUnsnappedPreferredSize() const override;
   base::string16 GetWindowTitle() const override;
   int GetDialogButtons() const override;
   void WindowClosing() override;
   bool ShouldUseCustomFrame() const override;
 
   views::WebView* web_view_;
 
   UserManagerProfileDialogDelegate* delegate_;
 
   std::unique_ptr<ScopedKeepAlive> keep_alive_;
   base::Time user_manager_started_showing_;
 
   base::FilePath signin_profile_path_;
 
   DISALLOW_COPY_AND_ASSIGN(UserManagerView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_PROFILES_USER_MANAGER_VIEW_H_