[Webkit-unassigned] [Bug 154654] New: [TreatNullAs=NullString] should be renamed to [TreatNullAs=EmptyString] to match Web IDL
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Feb 24 14:24:14 PST 2016
https://bugs.webkit.org/show_bug.cgi?id=154654
Bug ID: 154654
Summary: [TreatNullAs=NullString] should be renamed to
[TreatNullAs=EmptyString] to match Web IDL
Classification: Unclassified
Product: WebKit
Version: WebKit Nightly Build
Hardware: Unspecified
URL: http://heycam.github.io/webidl/#TreatNullAs
OS: Unspecified
Status: NEW
Severity: Normal
Priority: P2
Component: Bindings
Assignee: webkit-unassigned at lists.webkit.org
Reporter: cdumez at apple.com
CC: darin at apple.com, sam at webkit.org
[TreatNullAs=NullString] should be renamed to [TreatNullAs=EmptyString] to match Web IDL:
http://heycam.github.io/webidl/#TreatNullAs
The current WebKit behavior is the same as the IDL one [1], only the naming differs. We should align.
Note that [TreatNullAs=EmptyString] is a standard WebIDL extended attribute and is used in specifications. WebKit normally uses it in cases where the specification has it. However, WebKit also uses [TreatNullAs=NullString, TreatUndefined=NullString] for attributes / parameters that should actually be nullable. This is because support for nullable in our bindings generator is currently weak / incomplete. We should eventually use nullable (e.g. DOMString? value) [2] instead of [TreatNullAs=NullString, TreatUndefined=NullString] for those cases but this is a separate issue. Note that [TreatUndefined=XXX] is no longer part of Web IDL and should be dropped entirely once we better support nullable.
[1] http://heycam.github.io/webidl/#es-DOMString
[2] http://heycam.github.io/webidl/#idl-nullable-type
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160224/7769ed28/attachment.html>
More information about the webkit-unassigned
mailing list