Don't use a scoped_refptr for StringVar::FromPPVar

webkit/plugins/ppapi/message_channel.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/message_channel.h"
 
 #include <cstdlib>
 #include <string>
 
 #include "base/logging.h"
@@ skipping 51 matching lines @@
     case PP_VARTYPE_BOOL:
       BOOLEAN_TO_NPVARIANT(var.value.as_bool, *result);
       break;
     case PP_VARTYPE_INT32:
       INT32_TO_NPVARIANT(var.value.as_int, *result);
       break;
     case PP_VARTYPE_DOUBLE:
       DOUBLE_TO_NPVARIANT(var.value.as_double, *result);
       break;
     case PP_VARTYPE_STRING: {
-      scoped_refptr<StringVar> string(StringVar::FromPPVar(var));
+      StringVar* string = StringVar::FromPPVar(var);
       if (!string) {
         VOID_TO_NPVARIANT(*result);
         return false;
       }
       const std::string& value = string->value();
       STRINGN_TO_NPVARIANT(value.c_str(), value.size(), *result);
       break;
     }
     case PP_VARTYPE_OBJECT:
       // Objects are not currently supported.
@@ skipping 12 matching lines @@
 // PP_Var with the a copy of |var|'s string contents and a reference count of 1.
 //
 // TODO(dmichael):  We need to do structured clone eventually to copy a object
 // structure.  The details and PPAPI changes for this are TBD.
 PP_Var CopyPPVar(const PP_Var& var) {
   if (var.type == PP_VARTYPE_OBJECT) {
     // Objects are not currently supported.
     NOTIMPLEMENTED();
     return PP_MakeUndefined();
   } else if (var.type == PP_VARTYPE_STRING) {
-    scoped_refptr<StringVar> string(StringVar::FromPPVar(var));
+    StringVar* string = StringVar::FromPPVar(var);
     if (!string)
       return PP_MakeUndefined();
     return StringVar::StringToPPVar(string->pp_module(), string->value());
   } else {
     return var;
   }
 }
 
 //------------------------------------------------------------------------------
 // Implementations of NPClass functions.  These are here to:
@@ skipping 307 matching lines @@
   //   SetPassthroughObject(passthrough_object());
   if (passthrough_object_)
     WebBindings::releaseObject(passthrough_object_);
 
   passthrough_object_ = passthrough;
 }
 
 }  // namespace ppapi
 }  // namespace webkit