[Webkit-unassigned] [Bug 40835] Adds support for throwing behavior of SerializedScriptValue to be specified in the IDL.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jul 1 00:40:05 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=40835
Jeremy Orlow <jorlow at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #60111|review? |review-
Flag| |
--- Comment #34 from Jeremy Orlow <jorlow at chromium.org> 2010-07-01 00:40:05 PST ---
(From update of attachment 60111)
WebCore/bindings/js/SerializedScriptValue.h:179
+ static PassRefPtr<SerializedScriptValue> createWithoutThrowing(JSC::ExecState* exec, JSC::JSValue value);
don't need exec or value here...or below
WebCore/bindings/js/SerializedScriptValue.h:183
+ static PassRefPtr<SerializedScriptValue> create(JSC::ExecState* exec, JSC::JSValue value, ExceptionCode* ec);
don't need ec
WebCore/bindings/scripts/CodeGeneratorJS.pm:2234
+ push(@implContent, "\n" . NativeSerializedScriptValueToJSValue($function->signature, $functionString, @{$function->raisesExceptions})) if $function->signature->type eq "SerializedScriptValue";
what about any? be sure to test for this if it's true that you should have had "any"?
WebCore/bindings/scripts/CodeGeneratorJS.pm:2473
+ if ($type eq "SerializedScriptValue" or $type eq "any") {
Do you actually need this? Can't you assume it?
WebCore/bindings/scripts/test/JS/JSTestObj.cpp:832
+ SerializedScriptValue* serializedScriptValue = imp->serializedValueReturnRaises(ec);
Wouldn't this need to be a PassRefPtr?
Does anything even use this code yet? If not, then maybe we shouldn't test for it and/or worry about it.
WebCore/bindings/scripts/test/JS/JSTestObj.cpp:833
+ JSC::JSValue result = serializedScriptValue && !ec ? serializedScriptValue->deserialized() : jsNull();
if ec is set, we should return undefined. Otherwise null.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list