PageLoadMetrics renderer and browser implementation.

components/page_load_metrics/renderer/page_load_metrics_render_frame_observer.h

Index: components/page_load_metrics/renderer/page_load_metrics_render_frame_observer.h
diff --git a/components/page_load_metrics/renderer/page_load_metrics_render_frame_observer.h b/components/page_load_metrics/renderer/page_load_metrics_render_frame_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..c71d0c63b237dab8254184c3775e817e12d668ef
--- /dev/null
+++ b/components/page_load_metrics/renderer/page_load_metrics_render_frame_observer.h
@@ -0,0 +1,52 @@
+// 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 "components/page_load_metrics/common/page_load_timing.h"
+#include "content/public/renderer/render_frame_observer.h"
+
+namespace base {
+class Timer;
+}  // namespace base
+
+namespace page_load_metrics {
+
+class PageTimingMetricsSender;
+
+// MainRenderFrameObserver observes RenderFrame notifications, and
+// sends page load timing information to the browser process over IPC.
+class MainRenderFrameObserver : public content::RenderFrameObserver {
+ public:
+  explicit MainRenderFrameObserver(content::RenderFrame* render_frame);
+  ~MainRenderFrameObserver() override;
+
+  // RenderFrameObserver implementation
+  void OnStop() override;
+  void DidFinishDocumentLoad() override;
+  void DidFinishLoad() override;
+  void DidFailProvisionalLoad(const blink::WebURLError&) override;
+  void DidCommitProvisionalLoad(bool is_new_navigation,
+                                bool is_same_page_navigation) override;
+
+ protected:
+  virtual bool ShouldSendMetrics() const;

[Bryan McQuade, 2015/09/04 20:53:18]

I learned from your use of private virtual. If any of these can be moved to
being private, rather than protected (if they are only overridden in derived
classes, rather than called), I think that's worthwhile.

[Charlie Harrison, 2015/09/08 23:05:15]

Done.

+  virtual PageLoadTiming GetTiming() const;
+  virtual scoped_ptr<base::Timer> CreateTimer() const;
+
+ private:
+  // Will be null when we're not actively sending metrics.
+  scoped_ptr<PageTimingMetricsSender> page_timing_metrics_sender_;
+
+  void SendMetrics();
+
+  DISALLOW_COPY_AND_ASSIGN(MainRenderFrameObserver);
+};
+
+}  // namespace page_load_metrics
+
+#endif  // COMPONENTS_PAGE_LOAD_METRICS_RENDERER_PAGE_LOAD_METRICS_RENDER_FRAME_OBSERVER_H_