From internal discussion with sky et al.:
* Eliminate one-off GetXYZMetric()-type functions from ViewsDelegate, replace with the sorts of enum-based functions currently on LayoutDelegate
* Only expose enums values in ui/views/ that ui/views/ needs
* In the ChromeViewsDelegate, implement like:
class ChromeViewsDelegate : public ViewsDelegate {
enum ChromeMetric {
XXX = ViewsMetric::LAST_VALUE,
YYY,
}
int GetMetric(ChromeMetric metric) const;
private:
int GetMetric(ViewsMetric metric) const override {
return GetMetric(static_cast<ChromeMetric>(metric));
}
};
* Probably at this point we can eliminate LayoutDelegate entirely and move anything left on it to ChromeViewsDelegate
+CC some sanity-checkers
Comment 1 by bugdroid1@chromium.org
, Feb 1 2017