Title: [119027] trunk
- Revision
- 119027
- Author
- [email protected]
- Date
- 2012-05-30 19:58:48 -0700 (Wed, 30 May 2012)
Log Message
[JSC] SerializedScriptValue.create() succeeds even if MessagePort object cannot be found in transferred ports
https://bugs.webkit.org/show_bug.cgi?id=87118
Patch by Christophe Dumez <[email protected]> on 2012-05-30
Reviewed by Adam Barth.
Source/WebCore:
Make SerializedScriptValue.create() throw an exception if one of the
MessagePort objects cannot be found in the transferred ports. This
matches the behavior of the V8 implementation.
Test: webintents/web-intents-obj-constructor.html
* bindings/js/SerializedScriptValue.cpp:
(WebCore::CloneSerializer::dumpIfTerminal):
LayoutTests:
Unskip webintents/web-intents-obj-constructor.html now that
SerializedScriptValue.create() throws an exception if one of the
MessagePort objects cannot be found in the transferred ports.
* platform/efl/test_expectations.txt:
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (119026 => 119027)
--- trunk/LayoutTests/ChangeLog 2012-05-31 02:53:48 UTC (rev 119026)
+++ trunk/LayoutTests/ChangeLog 2012-05-31 02:58:48 UTC (rev 119027)
@@ -1,3 +1,16 @@
+2012-05-30 Christophe Dumez <[email protected]>
+
+ [JSC] SerializedScriptValue.create() succeeds even if MessagePort object cannot be found in transferred ports
+ https://bugs.webkit.org/show_bug.cgi?id=87118
+
+ Reviewed by Adam Barth.
+
+ Unskip webintents/web-intents-obj-constructor.html now that
+ SerializedScriptValue.create() throws an exception if one of the
+ MessagePort objects cannot be found in the transferred ports.
+
+ * platform/efl/test_expectations.txt:
+
2012-05-30 Jessie Berlin <[email protected]>
[Win] Need results for ietestcenter CSS3 tests
Modified: trunk/LayoutTests/platform/efl/test_expectations.txt (119026 => 119027)
--- trunk/LayoutTests/platform/efl/test_expectations.txt 2012-05-31 02:53:48 UTC (rev 119026)
+++ trunk/LayoutTests/platform/efl/test_expectations.txt 2012-05-31 02:58:48 UTC (rev 119027)
@@ -651,7 +651,6 @@
// Incomplete Web Intents support
BUGWK86866 SKIP : webintents/intent-tag.html = TEXT
BUGWK86865 SKIP : webintents/web-intents-delivery.html = TEXT
-BUGWK87118 : webintents/web-intents-obj-constructor.html = TEXT
// EFL's LayoutTestController does not implement setAutomaticLinkDetectionEnabled
BUGWK85463 SKIP : editing/inserting/typing-space-to-trigger-smart-link.html = FAIL
Modified: trunk/Source/WebCore/ChangeLog (119026 => 119027)
--- trunk/Source/WebCore/ChangeLog 2012-05-31 02:53:48 UTC (rev 119026)
+++ trunk/Source/WebCore/ChangeLog 2012-05-31 02:58:48 UTC (rev 119027)
@@ -1,3 +1,19 @@
+2012-05-30 Christophe Dumez <[email protected]>
+
+ [JSC] SerializedScriptValue.create() succeeds even if MessagePort object cannot be found in transferred ports
+ https://bugs.webkit.org/show_bug.cgi?id=87118
+
+ Reviewed by Adam Barth.
+
+ Make SerializedScriptValue.create() throw an exception if one of the
+ MessagePort objects cannot be found in the transferred ports. This
+ matches the behavior of the V8 implementation.
+
+ Test: webintents/web-intents-obj-constructor.html
+
+ * bindings/js/SerializedScriptValue.cpp:
+ (WebCore::CloneSerializer::dumpIfTerminal):
+
2012-05-30 Mark Pilgrim <[email protected]>
[Chromium] Call fileUtilities methods directly
Modified: trunk/Source/WebCore/bindings/js/SerializedScriptValue.cpp (119026 => 119027)
--- trunk/Source/WebCore/bindings/js/SerializedScriptValue.cpp 2012-05-31 02:53:48 UTC (rev 119026)
+++ trunk/Source/WebCore/bindings/js/SerializedScriptValue.cpp 2012-05-31 02:58:48 UTC (rev 119027)
@@ -634,7 +634,9 @@
write(index->second);
return true;
}
- return false;
+ // MessagePort object could not be found in transferred message ports
+ code = ValidationError;
+ return true;
}
if (obj->inherits(&JSArrayBuffer::s_info)) {
RefPtr<ArrayBuffer> arrayBuffer = toArrayBuffer(obj);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
