[Webkit-unassigned] [Bug 122301] New: URLMediaSource.idl and URLMediaStream.idl are wrong

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 3 14:48:21 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=122301

           Summary: URLMediaSource.idl and URLMediaStream.idl are wrong
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Media Elements
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: dchris at gmail.com
                CC: darin at apple.com, eric.carlson at apple.com,
                    jer.noble at apple.com


URLMediaSource.idl and URLMediaStream.idl are partial interfaces that extend the URL interface. However, this has no impact at the moment and it causes the bindings generator to generate a warning because the URL interface does not exist. The interface is called DOMURL in WebKit.

I assume this code was imported from Blink where the IDL interfaces' name matches their JS name (I tried to do the same in WebKit but Darin thought the IDL interfaces' name should be consistent with the implementation name instead).

You should update those partial interfaces as follows:
- partial interface URL {
+ partial interface DOMURL {

so that is actually does something and affects the behavior on JS side.

Jer Noble, could you please look into this since I believe you are working on this?

--
Bindings generator warning:
Use of uninitialized value $targetIdlFile in hash element at Source/WebCore/bindings/scripts/preprocess-idls.pl line 128.

We should probably have the script die instead in this case and print a useful error message to let the developer know its partial interface (or IDL implements statement is wrong) is wrong.

-- 
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