views: support dialog width snapping once and for all

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

 // Copyright 2016 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_HARMONY_CHROME_LAYOUT_PROVIDER_H_
 #define CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 48 matching lines @@
 class ChromeLayoutProvider : public views::LayoutProvider {
  public:
   ChromeLayoutProvider() {}
   ~ChromeLayoutProvider() override {}
 
   static ChromeLayoutProvider* Get();
   static std::unique_ptr<views::LayoutProvider> CreateLayoutProvider();
 
   int GetDistanceMetric(int metric) const override;
 
   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:
   //   ---------------------------
   //   | Label 1      Checkbox 1 |
   //   | Label 2      Checkbox 2 |
   //   ---------------------------
   // This value controls the alignment used for "Label 1" and "Label 2".
   virtual views::GridLayout::Alignment GetControlLabelGridAlignment() const;
 
   // Returns whether to use extra padding on dialogs. If this is false, content
   // Views for dialogs should not insert extra padding at their own edges.
   virtual bool UseExtraDialogPadding() const;
 
   // Returns whether to show the icon next to the title text on a dialog.
   virtual bool ShouldShowWindowIcon() const;
 
   // DEPRECATED.  Returns whether Harmony mode is enabled.
   //
   // Instead of using this, create a generic solution that works for all UI
   // types, e.g. by adding a new LayoutDistance value that means what you need.
   //
   // 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);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_HARMONY_CHROME_LAYOUT_PROVIDER_H_