PageLoadMetrics renderer and browser implementation (deprecated)

components/page_load_metrics/renderer/page_load_metrics_render_frame_observer.h

+// Copyright 2015 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 COMPONENTS_PAGE_LOAD_METRICS_RENDERER_PAGE_LOAD_METRICS_RENDER_FRAME_OBSERVER_H_
+#define COMPONENTS_PAGE_LOAD_METRICS_RENDERER_PAGE_LOAD_METRICS_RENDER_FRAME_OBSERVER_H_
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/time/time.h"
+#include "content/public/renderer/render_frame_observer.h"
+
+namespace page_load_metrics {
+
+// PageLoadMetricsRenderFrameObserver observes RenderFrame notifications, and
+// sends page load timing information to the browser process over IPC.
+class PageLoadMetricsRenderFrameObserver : public content::RenderFrameObserver {
+ public:
+  // The Timing interface provides access to the timing metrics we send to the
+  // browser process.
+  class Timing {
+   public:
+    Timing() {}
+    virtual ~Timing() {}
+    virtual base::Time GetNavigationStart() const = 0;
+    virtual base::Time GetDOMContentLoadedEvent() const = 0;
+    virtual base::Time GetLoadEvent() const = 0;
+    virtual base::Time GetFirstLayout() const = 0;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(Timing);
+  };
+
+  explicit PageLoadMetricsRenderFrameObserver(
+      content::RenderFrame* render_frame);
+  ~PageLoadMetricsRenderFrameObserver() override;
+
+  // RenderFrameObserver implementation
+  void DidFinishDocumentLoad() override;
+  void DidFinishLoad() override;
+

[Randy Smith (Not in Mondays), 2015/08/17 19:31:08]

I'm used to blank lines marking the end of // <baseclass> implementation blocks,
so I'd be inclined to remove this blank line.

[Bryan McQuade, 2015/08/25 13:53:36]

Done.

+  void DidStartProvisionalLoad() override;
+  void DidCommitProvisionalLoad(bool is_new_navigation,
+                                bool is_same_page_navigation) override;
+
+ protected:
+  virtual bool ShouldSendMetrics();

[Randy Smith (Not in Mondays), 2015/08/17 19:31:08]

Mark as const to indicate that it really doesn't change state?  (Which seems to
be how it's used; just bringing decoration in line with usage.)

[Bryan McQuade, 2015/08/25 13:53:35]

Done.

+  virtual scoped_ptr<Timing> CreateTiming();
+
+ private:
+  class PageMetricsSender;
+  scoped_ptr<PageMetricsSender> page_metrics_sender_;

[Randy Smith (Not in Mondays), 2015/08/17 19:31:08]

Suggestion: Comment here indicating when this pointer is null and non-null.

[Bryan McQuade, 2015/08/25 13:53:35]

Done.

+
+  void MaybeSendMetrics();
+
+  DISALLOW_COPY_AND_ASSIGN(PageLoadMetricsRenderFrameObserver);
+};
+
+}  // namespace page_load_metrics
+
+#endif  // COMPONENTS_PAGE_LOAD_METRICS_RENDERER_PAGE_LOAD_METRICS_RENDER_FRAME_OBSERVER_H_