Make 'any'-typed attributes of events available in isolated worlds

Source/bindings/v8/V8Binding.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 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 521 matching lines @@
     return MainWorld;
 }
 
 DOMWrapperWorld* isolatedWorldForIsolate(v8::Isolate* isolate)
 {
     V8PerIsolateData* data = V8PerIsolateData::from(isolate);
     if (data->workerDOMDataStore())
         return 0;
     if (!DOMWrapperWorld::isolatedWorldsExist())
         return 0;
-    ASSERT(!v8::Context::GetEntered().IsEmpty());
-    return DOMWrapperWorld::isolatedWorld(v8::Context::GetEntered());
+    ASSERT(v8::Context::InContext());
+    return DOMWrapperWorld::isolatedWorld(v8::Context::GetCurrent());

[haraken, 2013/07/18 00:00:29]

Why did you make this change?

I guess your new code is correct, but I'd like to understand why the old code
was using Context::GetEntered() instead of Context::GetCurrent().

[adamk, 2013/07/18 01:09:14]

I can't explain why the old code was using GetEntered (that was just a series of
refactors that landed us at the code you see today), but I needed to change it
to GetCurrent() for this patch because I thought it might be possible to trigger
the code without having Entered a context.

Note that, since an isolated world shouldn't be able to communicate with any
other, _any_ context on the stack of contexts should be able to tell us what
world we're in.

+}
+
+v8::Local<v8::Value> getHiddenValueFromMainWorldWrapper(v8::Isolate* isolate, ScriptWrappable* wrappable, v8::Handle<v8::String> key)
+{
+    v8::Local<v8::Object> wrapper = wrappable->newLocalWrapper(isolate);
+    return !wrapper.IsEmpty() ? wrapper->GetHiddenValue(key) : v8::Local<v8::Value>();

[haraken, 2013/07/18 00:00:29]

Nit: I'd prefer:

wrapper.Empty() ? v8::Local<v8::Value>() : wrapper->GetHiddenValue(key)

[adamk, 2013/07/18 01:09:14]

Will do on the next upload.

 }
 
 } // namespace WebCore