Broke out layout metric information from ViewsDelegate to LayoutProvider

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

-// Copyright 2016 The Chromium Authors. All rights reserved.
+// Copyright 2017 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_HARMONY_LAYOUT_DELEGATE_H_
 #define CHROME_BROWSER_UI_VIEWS_HARMONY_HARMONY_LAYOUT_DELEGATE_H_
 
-#include "chrome/browser/ui/views/harmony/layout_delegate.h"
+#include <memory>

[Peter Kasting, 2017/04/06 08:13:36]

Nit: Why this #include?

[kylix_rd, 2017/04/06 16:48:19]

Probably an over zealous copy operation as I initially populated this file.
Removed.

 
-class HarmonyLayoutDelegate : public LayoutDelegate {
+#include "base/macros.h"
+#include "chrome/browser/ui/views/harmony/chrome_layout_delegate.h"
+#include "ui/gfx/geometry/insets.h"
+#include "ui/views/layout/grid_layout.h"

[Peter Kasting, 2017/04/06 08:13:36]

Nit: insets.h and grid_layout.h shouldn't be necessary, even from an IWYU
perspective, since the only references to them are in overrides, and thus the
types must already be available (and if they disappear from the base class,
they'll necessarily disappear from here too, so this is pure maintenance
burden).

The IWYU clang folks agree with this, though they've had challenges
implementing; see
https://github.com/include-what-you-use/include-what-you-use/issues/105 .

[kylix_rd, 2017/04/06 16:48:19]

Good to know. It's a subtle exception to the IWYU rule.

+
+class HarmonyLayoutDelegate : public ChromeLayoutDelegate {
  public:
   // The Harmony layout unit. All distances are in terms of this unit.
   static constexpr int kHarmonyLayoutUnit = 16;
 
   HarmonyLayoutDelegate() {}
   ~HarmonyLayoutDelegate() override {}
 
-  // Returns the singleton HarmonyLayoutDelegate instance.
-  static HarmonyLayoutDelegate* Get();
-
-  // views::LayoutDelegate:
-  int GetMetric(Metric metric) const override;
+  gfx::Insets GetInsetsMetric(ChromeInsetsMetric metric) const override;
+  int GetDistanceMetric(ChromeDistanceMetric metric) const override;
   views::GridLayout::Alignment GetControlLabelGridAlignment() const override;
   bool UseExtraDialogPadding() const override;
+  bool ShouldShowWindowIcon() const override;
   bool IsHarmonyMode() const override;
   int GetDialogPreferredWidth(DialogWidth width) const override;
-  bool ShouldShowWindowIcon() const override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(HarmonyLayoutDelegate);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_HARMONY_HARMONY_LAYOUT_DELEGATE_H_