views: support dialog width snapping once and for all

chrome/browser/ui/views/harmony/chrome_layout_provider.h

Index: chrome/browser/ui/views/harmony/chrome_layout_provider.h
diff --git a/chrome/browser/ui/views/harmony/chrome_layout_provider.h b/chrome/browser/ui/views/harmony/chrome_layout_provider.h
index 1fe25cf55ac59827185aaba46a432b0d6cbcd230..f9e0564b2d02a94458a82c395fbc2dbf547006d8 100644
--- a/chrome/browser/ui/views/harmony/chrome_layout_provider.h
+++ b/chrome/browser/ui/views/harmony/chrome_layout_provider.h
@@ -68,6 +68,8 @@ class ChromeLayoutProvider : public views::LayoutProvider {
 
   const views::TypographyProvider& GetTypographyProvider() const override;

[Peter Kasting, 2017/04/19 19:26:13]

Nit: Let's just kill the blank lines between these override declarations (would
be nice to offset them with "// views::LayoutProvider:" as well)

[Elly Fong-Jones, 2017/04/20 19:53:59]

Done.

 
+  int GetSnappedDialogWidth(int min_width) const override;
+
   // Returns the alignment used for control labels in a GridLayout; for example,
   // in this GridLayout:
   //   ---------------------------
@@ -92,10 +94,6 @@ class ChromeLayoutProvider : public views::LayoutProvider {
   // TODO(pkasting): Fix callers and remove this.
   virtual bool IsHarmonyMode() const;
 
-  // Returns the preferred width in DIPs for a dialog of the specified |width|.
-  // May return 0 if the dialog has no preferred width.
-  virtual int GetDialogPreferredWidth(DialogWidth width) const;
-
  private:
   DISALLOW_COPY_AND_ASSIGN(ChromeLayoutProvider);
 };