Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(254)

Unified Diff: ppapi/shared_impl/var.cc

Issue 15806016: Update ppapi/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ppapi/shared_impl/tracked_callback.cc ('k') | ppapi/shared_impl/var_tracker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/shared_impl/var.cc
diff --git a/ppapi/shared_impl/var.cc b/ppapi/shared_impl/var.cc
index 0aff5837fe9a4a383061d9558c5c702bcebbbcc2..c872738cd3f80fd44f2e9ce787a33d4cc412433e 100644
--- a/ppapi/shared_impl/var.cc
+++ b/ppapi/shared_impl/var.cc
@@ -157,7 +157,7 @@ PP_Var StringVar::StringToPPVar(const std::string& var) {
// static
PP_Var StringVar::StringToPPVar(const char* data, uint32 len) {
scoped_refptr<StringVar> str(new StringVar(data, len));
- if (!str || !IsStringUTF8(str->value()))
+ if (!str.get() || !IsStringUTF8(str->value()))
return PP_MakeNull();
return str->GetPPVar();
}
@@ -168,7 +168,7 @@ StringVar* StringVar::FromPPVar(PP_Var var) {
return NULL;
scoped_refptr<Var> var_object(
PpapiGlobals::Get()->GetVarTracker()->GetVar(var));
- if (!var_object)
+ if (!var_object.get())
return NULL;
return var_object->AsStringVar();
}
@@ -202,7 +202,7 @@ ArrayBufferVar* ArrayBufferVar::FromPPVar(PP_Var var) {
return NULL;
scoped_refptr<Var> var_object(
PpapiGlobals::Get()->GetVarTracker()->GetVar(var));
- if (!var_object)
+ if (!var_object.get())
return NULL;
return var_object->AsArrayBufferVar();
}
« no previous file with comments | « ppapi/shared_impl/tracked_callback.cc ('k') | ppapi/shared_impl/var_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698