Move BrowserPluginDelegate's lifetime mgmt out of content/

extensions/renderer/guest_view/extensions_guest_view_container.h

Index: extensions/renderer/guest_view/extensions_guest_view_container.h
diff --git a/extensions/renderer/guest_view/extensions_guest_view_container.h b/extensions/renderer/guest_view/extensions_guest_view_container.h
index 4e3f3e9e5ec4c25f689afec81610b60f7bf4c7c7..f80da9148697596f5ee4ae48c259630f7079175e 100644
--- a/extensions/renderer/guest_view/extensions_guest_view_container.h
+++ b/extensions/renderer/guest_view/extensions_guest_view_container.h
@@ -19,7 +19,6 @@ namespace extensions {
 class ExtensionsGuestViewContainer : public guest_view::GuestViewContainer {
  public:
   explicit ExtensionsGuestViewContainer(content::RenderFrame* render_frame);
-  ~ExtensionsGuestViewContainer() override;
 
   void RegisterDestructionCallback(v8::Local<v8::Function> callback,
                                    v8::Isolate* isolate);
@@ -29,9 +28,14 @@ class ExtensionsGuestViewContainer : public guest_view::GuestViewContainer {
   // BrowserPluginDelegate implementation.
   void DidResizeElement(const gfx::Size& new_size) override;
 
+ protected:
+  ~ExtensionsGuestViewContainer() override;
+
  private:
   void CallElementResizeCallback(const gfx::Size& new_size);
 
+  void OnDestroy() override;

[Fady Samuel, 2015/06/05 17:31:13]

What is this overriding. Please specify.

[lazyboy, 2015/06/05 21:24:00]

Done.

+
   v8::Global<v8::Function> destruction_callback_;
   v8::Isolate* destruction_isolate_;