Modified RenderFrameObserver API

content/public/renderer/render_frame_observer.h

Index: content/public/renderer/render_frame_observer.h
diff --git a/content/public/renderer/render_frame_observer.h b/content/public/renderer/render_frame_observer.h
index 1e3c6717945067d9aacb6ba5b855d97782e443e9..6afb8a7e75d1cddd303839a44a672a5c83dbe8c8 100644
--- a/content/public/renderer/render_frame_observer.h
+++ b/content/public/renderer/render_frame_observer.h
@@ -102,6 +102,9 @@ class CONTENT_EXPORT RenderFrameObserver : public IPC::Listener,
   // Called when the focused node has changed to |node|.
   virtual void FocusedNodeChanged(const blink::WebNode& node) {}
 
+  // Notifications when Performance data becomes available

[Bryan McQuade, 2015/08/14 12:01:41]

for the comment, I'd reference the class name explicitly:
// Notifications when |PerformanceTiming| data becomes available.

+  virtual void OnPerformanceTimingChanged() {}

[Bryan McQuade, 2015/08/14 12:01:40]

in keeping with the other naming, maybe call this DidChangePerformanceTiming?

[Bryan McQuade, 2015/08/14 12:01:41]

In addition to this change, I'd also include the part of the CL where
RenderFrameImpl implements the new callback and invokes this new method on all
of its observers.

+
   // IPC::Listener implementation.
   bool OnMessageReceived(const IPC::Message& message) override;