Add Startup.BrowserView.FirstPaint / .CompositingEnded histograms.

chrome/browser/ui/views/frame/browser_view_histogram_helper.h

+// 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_FRAME_BROWSER_VIEW_HISTOGRAM_HELPER_H_
+#define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_HISTOGRAM_HELPER_H_
+
+#include "base/macros.h"
+#include "base/time/time.h"
+#include "ui/compositor/compositor_observer.h"
+
+// Class that encapsulates logic of recording BrowserView.Paint-histograms.
+class BrowserViewHistogramHelper : public ui::CompositorObserver {
+ public:
+  BrowserViewHistogramHelper();

[Peter Kasting, 2017/03/24 22:39:18]

Nit: Feel free to use =default

[themblsha, 2017/03/31 14:32:31]

Seems to be no longer applicable with ScopedObserver in place.

+  ~BrowserViewHistogramHelper();
+
+  void OnDidPaintChildren(ui::Compositor* compositor);
+
+ private:
+  void RemoveCompositorObserver();
+
+  // ui::CompositorObserver:
+  void OnCompositingDidCommit(ui::Compositor* compositor) override {}
+  void OnCompositingStarted(ui::Compositor* compositor,
+                            base::TimeTicks start_time) override {}
+  void OnCompositingEnded(ui::Compositor* compositor) override;
+  void OnCompositingLockStateChanged(ui::Compositor* compositor) override {}
+  void OnCompositingShuttingDown(ui::Compositor* compositor) override;
+
+  ui::Compositor* compositor_ = nullptr;
+

[Peter Kasting, 2017/03/24 22:39:18]

Nit: Blank line unnecessary

[themblsha, 2017/03/31 14:32:31]

Done.

+  bool did_first_paint_ = false;
+
+  DISALLOW_COPY_AND_ASSIGN(BrowserViewHistogramHelper);
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_HISTOGRAM_HELPER_H_