Ensure that constructor functions are created in the context of the...

webkit/port/bindings/v8/v8_proxy.cpp

 // Copyright (c) 2008, Google Inc.
 // All rights reserved.
 // 
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 // 
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 1442 matching lines @@
 {
     ASSERT(ContextInitialized());
     v8::Local<v8::Value> cached =
         m_dom_constructor_cache->Get(v8::Integer::New(V8ClassIndex::ToInt(t)));
     if (cached->IsFunction()) {
         return v8::Local<v8::Function>::Cast(cached);
     }
 
     // Not in cache.
     {
+        // Enter the context of the proxy to make sure that the
+        // function is constructed in the context corresponding to
+        // this proxy.
+        v8::Context::Scope scope(m_context);
         v8::Handle<v8::FunctionTemplate> templ = GetTemplate(t);
+        // Getting the function might fail if we're running out of
+        // stack or memory.
         v8::TryCatch try_catch;
-        // This might fail if we're running out of stack or memory.
         v8::Local<v8::Function> value = templ->GetFunction();
         if (value.IsEmpty())
             return v8::Local<v8::Function>();
         m_dom_constructor_cache->Set(v8::Integer::New(t), value);
         // Hotmail fix, see comments in v8_proxy.h above
         // m_dom_constructor_cache.
         value->Set(v8::String::New("__proto__"), m_object_prototype);
         return value;
     }
 }
@@ skipping 293 matching lines @@
     // JS world, but we setup the constructor function lazily in
     // WindowNamedPropertyHandler::get.
     case V8ClassIndex::DOMPARSER:
       desc->SetCallHandler(USE_CALLBACK(DOMParserConstructor));
       break;
     case V8ClassIndex::XMLSERIALIZER:
       desc->SetCallHandler(USE_CALLBACK(XMLSerializerConstructor));
       break;
     case V8ClassIndex::XMLHTTPREQUEST: {
       // Reserve one more internal field for keeping event listeners.
-        v8::Local<v8::ObjectTemplate> instance_template =
-            desc->InstanceTemplate();
-        instance_template->SetInternalFieldCount(
-            V8Custom::kXMLHttpRequestInternalFieldCount);
-        desc->SetCallHandler(USE_CALLBACK(XMLHttpRequestConstructor));
-        break;
+      v8::Local<v8::ObjectTemplate> instance_template =
+          desc->InstanceTemplate();
+      instance_template->SetInternalFieldCount(
+          V8Custom::kXMLHttpRequestInternalFieldCount);
+      desc->SetCallHandler(USE_CALLBACK(XMLHttpRequestConstructor));
+      break;
     }
     case V8ClassIndex::XMLHTTPREQUESTUPLOAD: {
       // Reserve one more internal field for keeping event listeners.
       v8::Local<v8::ObjectTemplate> instance_template =
           desc->InstanceTemplate();
       instance_template->SetInternalFieldCount(
           V8Custom::kXMLHttpRequestInternalFieldCount);
       break;
     }
     case V8ClassIndex::XPATHEVALUATOR:
@@ skipping 1663 matching lines @@
     v8::Handle<v8::Function> frame_source_name;
     frame_source_name = v8::Local<v8::Function>::Cast(
         utility_context->Global()->Get(v8::String::New("frame_source_name")));
     if (frame_source_name.IsEmpty()) {
         return String();
     }
     return ToWebCoreString(v8::Debug::Call(frame_source_name));
 }
 
 }  // namespace WebCore