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

Source/bindings/v8/Dictionary.cpp

 /*
  * Copyright (C) 2010 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 160 matching lines @@
 bool Dictionary::get(const String& key, ScriptValue& value) const
 {
     v8::Local<v8::Value> v8Value;
     if (!getKey(key, v8Value))
         return false;
 
     value = ScriptValue(v8Value);
     return true;
 }
 
+bool Dictionary::get(const String& key, RefPtr<SerializedScriptValue>& value) const
+{
+    v8::Local<v8::Value> v8Value;
+    if (!getKey(key, v8Value))
+        return false;
+
+    value = SerializedScriptValue::createSwallowExceptions(v8Value, m_isolate);

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

Would you elaborate on why you want to ignore exceptions here?

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

Note that this file is now gone.

But in all the other callers, the reason is that this serialization should be as
invisible as possible to script. The whole patch is an attempt to simulate old
behavior (where the detail/state/data properties were directly accessible from
different worlds), and anything that makes the copying noticeable (beyond the
necessities, like calling getters on the object to-be-copied) should be avoided.

None of this behavior is specced; it's all an attempt to make extensions behave
sanely.

+    return true;
+}
+
 bool Dictionary::get(const String& key, unsigned short& value) const
 {
     v8::Local<v8::Value> v8Value;
     if (!getKey(key, v8Value))
         return false;
 
     v8::Local<v8::Int32> v8Int32 = v8Value->ToInt32();
     if (v8Int32.IsEmpty())
         return false;
     value = static_cast<unsigned short>(v8Int32->Value());
@@ skipping 408 matching lines @@
         v8::Local<v8::String> key = properties->Get(i)->ToString();
         if (!options->Has(key))
             continue;
         names.append(toWebCoreString(key));
     }
 
     return true;
 }
 
 } // namespace WebCore