Remove support for NPObjects.

content/renderer/pepper/pepper_webplugin_impl.cc

Index: content/renderer/pepper/pepper_webplugin_impl.cc
diff --git a/content/renderer/pepper/pepper_webplugin_impl.cc b/content/renderer/pepper/pepper_webplugin_impl.cc
index 7c4e04de0c9078c8157b46b465c45ad72e6e4640..98460629a28f832898091ee3d446b24f66d28547 100644
--- a/content/renderer/pepper/pepper_webplugin_impl.cc
+++ b/content/renderer/pepper/pepper_webplugin_impl.cc
@@ -23,7 +23,6 @@
 #include "third_party/WebKit/public/platform/WebRect.h"
 #include "third_party/WebKit/public/platform/WebSize.h"
 #include "third_party/WebKit/public/platform/WebURLLoaderClient.h"
-#include "third_party/WebKit/public/web/WebBindings.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebElement.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
@@ -97,9 +96,6 @@ bool PepperWebPluginImpl::initialize(WebPluginContainer* container) {
   if (!instance_.get())
     return false;
 
-  // Enable script objects for this plugin.
-  container->allowScriptObjects();

[piman, 2016/03/23 17:28:02]

To confirm, this doesn't affect PPB_Var_Deprecated, right?
It's probably leftover from when we used to implement it through NPObject
instead of v8 directly.

[dcheng, 2016/03/24 03:14:08]

I don't think so: in any case, every implementation of this method is empty, so
removing it should be a no-op.

-
   auto weak_this = weak_factory_.GetWeakPtr();
   bool success =
       instance_->Initialize(init_data_->arg_names, init_data_->arg_values,
@@ -150,12 +146,6 @@ bool PepperWebPluginImpl::initialize(WebPluginContainer* container) {
 }
 
 void PepperWebPluginImpl::destroy() {
-  // Tell |container_| to clear references to this plugin's script objects.
-  if (container_)
-    container_->clearScriptObjects();
-
-  container_ = nullptr;
-
   if (instance_.get()) {
     ppapi::PpapiGlobals::Get()->GetVarTracker()->ReleaseVar(instance_object_);
     instance_object_ = PP_MakeUndefined();