Switch ContentSettingsObserver to be a RenderFrameObserver instead of a RenderViewObserver

content/renderer/render_frame_impl.h

Index: content/renderer/render_frame_impl.h
===================================================================
--- content/renderer/render_frame_impl.h	(revision 240912)
+++ content/renderer/render_frame_impl.h	(working copy)
@@ -62,6 +62,10 @@
   // Returns the RenderWidget associated with this frame.
   RenderWidget* GetRenderWidget();
 
+  // Called by RenderView right after creating this object when the
+  // blink::WebFrame has been created.
+  void MainWebFrameCreated(blink::WebFrame* frame);

[nasko, 2013/12/16 19:47:37]

Will we need this longer term? If not, might be useful to put a TODO to ensure
we don't miss it later on.

[jam, 2013/12/16 20:26:48]

this will be needed because when the RenderFrame is created, we don't have a
WebFrame yet. So it needs to be told so that it can tell its observers. this is
independent of whether a RF is per WF or per many.

+
 #if defined(ENABLE_PLUGINS)
   // Notification that a PPAPI plugin has been created.
   void PepperPluginCreated(RendererPpapiHost* host);