65787 2011-08-05 13:07:40 -0700 Add missing tests for LegacyDefaultOptionalArgument removal 2011-08-15 09:37:20 -0700 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 65802 65810 65812 65814 65815 65816 66234 66235 66236 1 abarth webkit-unassigned donggwan.kim pilgrim sam oldest_to_newest 447380 0 abarth 2011-08-05 13:07:40 -0700 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. 447504 1 sam 2011-08-05 17:39:07 -0700 This doesn't seem to include the changes to addEventListener/removeEventListener. Are those covered elsewhere? 447511 2 abarth 2011-08-05 17:46:08 -0700 > Are those covered elsewhere? Filed as blocking this bug. 447638 3 abarth 2011-08-06 12:35:45 -0700 > 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.