Created attachment 311353 [details] Patch

Comment on attachment 311353 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=311353&action=review > LayoutTests/webrtc/video-replace-track-to-null.html:24 > + framesEncoded = statItem.framesEncoded; Should this break? If not, what does it meant to have more than one?

Comment on attachment 311353 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=311353&action=review >> LayoutTests/webrtc/video-replace-track-to-null.html:24 >> + framesEncoded = statItem.framesEncoded; > > Should this break? If not, what does it meant to have more than one? We are in forEach, not in a for loop, I do not know whether there is a way to break here. There should only be one outbound-rtp since we have only one track. It might be good to ensure that in video-stats.html

Comment on attachment 311353 [details] Patch Rejecting attachment 311353 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 311353, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: le LayoutTests/TestExpectations.rej patching file LayoutTests/platform/mac-wk2/TestExpectations patching file LayoutTests/webrtc/audio-replace-track.html patching file LayoutTests/webrtc/video-replace-track-expected.txt patching file LayoutTests/webrtc/video-replace-track-to-null.html patching file LayoutTests/webrtc/video-replace-track.html Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Eric Carlson']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/3842443

<rdar://problem/34118218>

Created attachment 321124 [details] Patch

Comment on attachment 321124 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=321124&action=review > LayoutTests/TestExpectations:802 > -webkit.org/b/170704 webrtc/video-mute.html [ Failure ] > +webkit.org/b/170704 webrtc/video-mute.html [ Pass Failure ] Why?

(In reply to Alex Christensen from comment #7) > Comment on attachment 321124 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=321124&action=review > > > LayoutTests/TestExpectations:802 > > -webkit.org/b/170704 webrtc/video-mute.html [ Failure ] > > +webkit.org/b/170704 webrtc/video-mute.html [ Pass Failure ] > > Why? The line should be removed. I am keeping Failure expectation until I am sure it is not flaky.

Comment on attachment 321124 [details] Patch Rejecting attachment 321124 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 321124, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: hing file LayoutTests/webrtc/routines.js patching file LayoutTests/webrtc/video-mute.html patching file LayoutTests/webrtc/video-replace-muted-track.html patching file LayoutTests/webrtc/video-replace-track-expected.txt patching file LayoutTests/webrtc/video-replace-track-to-null.html patching file LayoutTests/webrtc/video-replace-track.html Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Eric Carlson']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/4605147

Created attachment 321331 [details] Patch for landing

Comment on attachment 321331 [details] Patch for landing Clearing flags on attachment: 321331 Committed r222273: <http://trac.webkit.org/changeset/222273>

All reviewed patches have been landed. Closing bug.