content/ fixups for scoped_refptr operator T* removal.

content/renderer/pepper/ppb_var_deprecated_impl.cc

Index: content/renderer/pepper/ppb_var_deprecated_impl.cc
diff --git a/content/renderer/pepper/ppb_var_deprecated_impl.cc b/content/renderer/pepper/ppb_var_deprecated_impl.cc
index a07127781dca7286aa24c776765eb2b28204f3ea..d8ac026ed9d865a216ffd460c5d28201a96d86e3 100644
--- a/content/renderer/pepper/ppb_var_deprecated_impl.cc
+++ b/content/renderer/pepper/ppb_var_deprecated_impl.cc
@@ -40,7 +40,7 @@ const char kUnableToCallMethodException[] = "Error: Unable to call method";
 class ObjectAccessor {
  public:
   ObjectAccessor(PP_Var var)
-      : object_var_(V8ObjectVar::FromPPVar(var)),
+      : object_var_(V8ObjectVar::FromPPVar(var).get()),

[dcheng, 2014/09/02 23:53:41]

Note: I'm not sure if this is safe or not. I've cced raymes@chromium.org to
review this particular line--usually, fields should probably be a scoped_refptr,
unless there's something that guarantees that ObjectAccessor will outlive the
V8ObjectVar.

[raymes, 2014/09/03 00:02:25]

This is safe. You might hit a conflict because this change got reverted again.
Sorry :(

[dcheng, 2014/09/03 00:04:20]

Ha. OK, thanks. I'll pre-emptively revert this change.

         instance_(object_var_ ? object_var_->instance() : NULL) {
   }