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

Source/bindings/v8/custom/V8CustomEventCustom.cpp

Index: Source/bindings/v8/custom/V8CustomEventCustom.cpp
diff --git a/Source/bindings/v8/custom/V8CustomEventCustom.cpp b/Source/bindings/v8/custom/V8CustomEventCustom.cpp
index d03eb673ddfdd238e5d9d7ed20aeedc7f7f1d504..270d5a1ab6d3b6d62a8629820947585f4d99bae2 100644
--- a/Source/bindings/v8/custom/V8CustomEventCustom.cpp
+++ b/Source/bindings/v8/custom/V8CustomEventCustom.cpp
@@ -34,7 +34,7 @@
 #include "V8Event.h"
 #include "bindings/v8/Dictionary.h"
 #include "bindings/v8/ScriptState.h"
-#include "bindings/v8/ScriptValue.h"
+#include "bindings/v8/SerializedScriptValue.h"
 #include "bindings/v8/V8Binding.h"
 #include "bindings/v8/V8DOMWrapper.h"
 #include "bindings/v8/V8HiddenPropertyName.h"
@@ -63,9 +63,14 @@ void V8CustomEvent::detailAttrGetterCustom(v8::Local<v8::String> name, const v8:
         return;
     }
 
-    SerializedScriptValue* serialized = event->serializedScriptValue();
-    if (serialized) {
-        result = serialized->deserialize();
+    if (!event->serializedScriptValue()) {
+        v8::Local<v8::Value> mainWorldDetail = getHiddenValueFromMainWorldWrapper(info.GetIsolate(), event, V8HiddenPropertyName::detail());
+        if (!mainWorldDetail.IsEmpty())
+            event->setSerializedDetail(SerializedScriptValue::createSwallowExceptions(mainWorldDetail, info.GetIsolate()));

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

What happens if SerializedScriptValue::createSwallowExceptions() return 0 ?

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

I don't believe that's possible. Is it? Even if it were, though, the if
statement below would fail nicely and we'd end up with v8::Null as our answer.

[haraken, 2013/07/18 01:28:01]

I thought it might be possible in a situation where createSwallowException
throws an exception (but he in fact doesn't throw), but anyway the situation
will be nicely handled in the below v8::Null().

+    }
+
+    if (event->serializedScriptValue()) {
+        result = event->serializedScriptValue()->deserialize();

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

Would it be possible to avoid deserializing the SerializedScriptValue that has
been just serialized in the line 69 ?

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

The entire point of the patch is to force a serialize/deserialize. See the CL
description, and bug https://code.google.com/p/chromium/issues/detail?id=85158
for more detail.

[haraken, 2013/07/18 01:28:01]

sorry, I was confused. Now got it! (You might want to add a comment about it.)

[adamk, 2013/07/18 16:48:32]

I think it's my fault, this CL came out of an in-person discussion with abarth
where we decided what the semantics would be. I didn't want to add comments
initially since I'd need to put it in three places. But now I think I ought to
just do that. Will comment before landing, thanks for the pair of fresh eyes.

         v8SetReturnValue(info, cacheState(info.Holder(), result));
         return;
     }
@@ -83,8 +88,13 @@ void V8CustomEvent::initCustomEventMethodCustom(const v8::FunctionCallbackInfo<v
     V8TRYCATCH_VOID(bool, cancelableArg, args[2]->BooleanValue());
     v8::Handle<v8::Value> detailsArg = args[3];
 
-    args.Holder()->SetHiddenValue(V8HiddenPropertyName::detail(), detailsArg);
     event->initEvent(typeArg, canBubbleArg, cancelableArg);
+
+    if (!detailsArg.IsEmpty()) {
+        args.Holder()->SetHiddenValue(V8HiddenPropertyName::detail(), detailsArg);
+        if (isolatedWorldForIsolate(args.GetIsolate()))
+            event->setSerializedDetail(SerializedScriptValue::createSwallowExceptions(detailsArg, args.GetIsolate()));
+    }
 }
 
 } // namespace WebCore