[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