Ensure we don't leak ObjectProxy objects

content/renderer/pepper/plugin_object.cc

Index: content/renderer/pepper/plugin_object.cc
diff --git a/content/renderer/pepper/plugin_object.cc b/content/renderer/pepper/plugin_object.cc
index cb5ff6af9bbd924ae918b87e196c2b130afb5cc6..3d122318ec9b594138b9c38dc95eeb6335f1b2ad 100644
--- a/content/renderer/pepper/plugin_object.cc
+++ b/content/renderer/pepper/plugin_object.cc
@@ -49,10 +49,9 @@ const char kInvalidValueException[] = "Error: Invalid value";
 // PluginObject ----------------------------------------------------------------
 
 PluginObject::~PluginObject() {
-  if (instance_) {
-    ppp_class_->Deallocate(ppp_class_data_);
+  ppp_class_->Deallocate(ppp_class_data_);

[piman, 2016/03/29 18:28:51]

So, in theory, we are not supposed to call back into the plugin after the
instance is deleted. Would it make sense instead to call Deallocate from
PluginObject::InstanceDeleted? Also that way I don't think you need the changes
in ObjectProxy.

+  if (instance_)
     instance_->RemovePluginObject(this);
-  }
 }
 
 // static