Bug 65787 - Add missing tests for LegacyDefaultOptionalArgument removal
: Add missing tests for LegacyDefaultOptionalArgument removal
Status: NEW
: WebKit
WebCore JavaScript
: 528+ (Nightly build)
: Unspecified Unspecified
: P2 Normal
Assigned To:
:
:
: 65802 65810 65812 65814 65815 65816 66234 66235 66236
:
  Show dependency treegraph
 
Reported: 2011-08-05 13:07 PST by
Modified: 2011-08-15 09:37 PST (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2011-08-05 13:07:40 PST
Looks like these patches have some behavior changes without tests.

https://bugs.webkit.org/show_bug.cgi?id=64539
https://bugs.webkit.org/show_bug.cgi?id=65338
https://bugs.webkit.org/show_bug.cgi?id=65353
https://bugs.webkit.org/show_bug.cgi?id=65355

Geolocation.clearWatch()
PositionCallback.handleEvent()
PositionErrorCallback.handleEvent()

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

SpeechInputResultList.item()

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

WebKitAnimationList.item()

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

MediaStreamList.item()
MediaStreamTrackList.item()
TouchList.item()

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

AudioBufferSourceNode.noteOn()
AudioBufferSourceNode.noteGrainOn()
AudioBufferSourceNode.noteOff()

We should add tests for the changed behavior.
------- Comment #1 From 2011-08-05 17:39:07 PST -------
This doesn't seem to include the changes to addEventListener/removeEventListener.  Are those covered elsewhere?
------- Comment #2 From 2011-08-05 17:46:08 PST -------
> Are those covered elsewhere?

Filed as blocking this bug.
------- Comment #3 From 2011-08-06 12:35:45 PST -------
> MediaStreamList.item()
> MediaStreamTrackList.item()

As far as I can tell, the PeerConnection API isn't far enough along to test this IDL change.  I don't see any tests that even instantiate the PeerConnection object.  It seems like the compat impact from this change is non-existent, so matching the spec is the best course of action.