Don't use a scoped_refptr for StringVar::FromPPVar

webkit/plugins/ppapi/ppapi_plugin_instance.cc

 // Copyright (c) 2011 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.
 
 #include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
@@ skipping 586 matching lines @@
 }
 
 string16 PluginInstance::GetSelectedText(bool html) {
   // Keep a reference on the stack. See NOTE above.
   scoped_refptr<PluginInstance> ref(this);
   if (!LoadSelectionInterface())
     return string16();
 
   PP_Var rv = plugin_selection_interface_->GetSelectedText(pp_instance(),
                                                            PP_FromBool(html));
-  scoped_refptr<StringVar> string(StringVar::FromPPVar(rv));
+  StringVar* string = StringVar::FromPPVar(rv);
   // Release the ref the plugin transfered to us.
   ResourceTracker::Get()->GetVarTracker()->ReleaseVar(rv);
   if (!string)
     return string16();
   return UTF8ToUTF16(string->value());
 }
 
 string16 PluginInstance::GetLinkAtPosition(const gfx::Point& point) {
   // Keep a reference on the stack. See NOTE above.
   scoped_refptr<PluginInstance> ref(this);
   if (!LoadPdfInterface())
     return string16();
 
   PP_Point p;
   p.x = point.x();
   p.y = point.y();
   PP_Var rv = plugin_pdf_interface_->GetLinkAtPosition(pp_instance(), p);
-  scoped_refptr<StringVar> string(StringVar::FromPPVar(rv));
+  StringVar* string = StringVar::FromPPVar(rv);
   // Release the ref the plugin transfered to us.
   ResourceTracker::Get()->GetVarTracker()->ReleaseVar(rv);
   if (!string)
     return string16();
   return UTF8ToUTF16(string->value());
 }
 
 void PluginInstance::Zoom(double factor, bool text_only) {
   // Keep a reference on the stack. See NOTE above.
   scoped_refptr<PluginInstance> ref(this);
@@ skipping 791 matching lines @@
 }
 
 PP_Var PluginInstance::ExecuteScript(PP_Instance instance,
                                      PP_Var script,
                                      PP_Var* exception) {
   TryCatch try_catch(module()->pp_module(), exception);
   if (try_catch.has_exception())
     return PP_MakeUndefined();
 
   // Convert the script into an inconvenient NPString object.
-  scoped_refptr<StringVar> script_string(StringVar::FromPPVar(script));
+  StringVar* script_string = StringVar::FromPPVar(script);
   if (!script_string) {
     try_catch.SetException("Script param to ExecuteScript must be a string.");
     return PP_MakeUndefined();
   }
   NPString np_script;
   np_script.UTF8Characters = script_string->value().c_str();
   np_script.UTF8Length = script_string->value().length();
 
   // Get the current frame to pass to the evaluate function.
   WebFrame* frame = container_->element().document().frame();
@@ skipping 94 matching lines @@
 }
 
 void PluginInstance::DoSetCursor(WebCursorInfo* cursor) {
   cursor_.reset(cursor);
   if (fullscreen_container_)
     fullscreen_container_->DidChangeCursor(*cursor);
 }
 
 }  // namespace ppapi
 }  // namespace webkit