Always use the caller's context in PepperTryCatchV8

content/renderer/pepper/pepper_try_catch.cc

 // Copyright 2014 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 "content/renderer/pepper/pepper_try_catch.h"
 
 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
 #include "gin/converter.h"
 #include "ppapi/shared_impl/ppapi_globals.h"
 #include "ppapi/shared_impl/var_tracker.h"
@@ skipping 52 matching lines @@
 
 PepperTryCatchV8::PepperTryCatchV8(
     PepperPluginInstanceImpl* instance,
     V8VarConverter::AllowObjectVars convert_objects,
     v8::Isolate* isolate)
     : PepperTryCatch(instance, convert_objects),
       exception_(PP_MakeUndefined()) {
   // Typically when using PepperTryCatchV8 we are passed an isolate. We verify
   // that this isolate is the same as the plugin isolate.
   DCHECK(isolate == instance_->GetIsolate());
-  // We assume we are already in the plugin context for PepperTryCatchV8.
-  DCHECK(GetContext() == isolate->GetCurrentContext());
+
+  // We assume that a handle scope and context has been setup by the user of
+  // this class. This is typically true because this class is used when calling
+  // into the plugin from JavaScript. We want to use whatever v8 context the
+  // caller is in.
 }
 
 PepperTryCatchV8::~PepperTryCatchV8() {
   ppapi::PpapiGlobals::Get()->GetVarTracker()->ReleaseVar(exception_);
 }
 
 bool PepperTryCatchV8::HasException() {
   return GetContext().IsEmpty() || exception_.type != PP_VARTYPE_UNDEFINED;
 }
 
@@ skipping 74 matching lines @@
   if (exception_is_set_) {
     NOTREACHED();
     return;
   }
   if (exception_)
     *exception_ = ppapi::StringVar::StringToPPVar(message, strlen(message));
   exception_is_set_ = true;
 }
 
 }  // namespace content