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.
This doesn't seem to include the changes to addEventListener/removeEventListener. Are those covered elsewhere?
> Are those covered elsewhere? Filed as blocking this bug.
> 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.