Bug 101751 – MediaStream API: Schedule the RTCDataChannel events to be triggered at idle state

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Tommy Widenflycht]

MediaStream API: Schedule the RTCDataChannel events to be triggered at idle state

[Tommy Widenflycht]

Created attachment 173290 [details]
Patch

[WebKit Review Bot]

Please wait for approval from [e-mail], [e-mail], [e-mail], [e-mail] or [e-mail] before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.

[Adam Barth]

Comment on attachment 173290 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=173290&action=review

> Source/WebCore/Modules/mediastream/RTCDataChannel.cpp:288
> +    events.clear();

This line shouldn't be needed.  The vector will get cleared naturally as the stack unwinds.

[Tommy Widenflycht]

Created attachment 173597 [details]
Patch for landing

[Tommy Widenflycht]

Comment on attachment 173290 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=173290&action=review

>> Source/WebCore/Modules/mediastream/RTCDataChannel.cpp:288
>> +    events.clear();
> 
> This line shouldn't be needed.  The vector will get cleared naturally as the stack unwinds.

Removed.

[WebKit Review Bot]

Comment on attachment 173597 [details]
Patch for landing

Clearing flags on attachment: 173597

Committed r134207: <http://trac.webkit.org/changeset/134207>