RenderFrame: flesh out Observer

content/public/renderer/render_frame.h

 // Copyright 2013 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 CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
 #define CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
 
 #include "content/common/content_export.h"
 #include "ipc/ipc_listener.h"
 #include "ipc/ipc_sender.h"
@@ skipping 59 matching lines @@
       blink::WebFrame* frame,
       const WebPluginInfo& info,
       const blink::WebPluginParams& params) = 0;
 
   // The client should handle the navigation externally.
   virtual void LoadURLExternally(
       blink::WebFrame* frame,
       const blink::WebURLRequest& request,
       blink::WebNavigationPolicy policy) = 0;
 
+  // OnStop notification from RenderView.
+  virtual void OnStop() = 0;

[nasko, 2014/01/28 20:41:39]

This doesn't need to be public, it is just an IPC endpoint for receiving
messages.

[Elly Fong-Jones, 2014/01/28 21:34:20]

Yes it does - it is not a "real" IPC endpoint but is instead called by
RenderViewObserver.

+
  protected:
   virtual ~RenderFrame() {}
 
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameImpl;
   RenderFrame() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_