RESOLVED FIXED 200975
Remove MediaStreamPrivate::scheduleDeferredTask
https://bugs.webkit.org/show_bug.cgi?id=200975
Summary Remove MediaStreamPrivate::scheduleDeferredTask
youenn fablet
Reported 2019-08-21 05:46:18 PDT
Remove MediaStreamPrivate::scheduleDeferredTask
Attachments
Patch (4.29 KB, patch)
2019-08-21 06:06 PDT, youenn fablet
no flags
Patch (16.03 KB, patch)
2019-08-23 07:35 PDT, youenn fablet
no flags
Patch (16.07 KB, patch)
2019-08-26 01:41 PDT, youenn fablet
no flags
Patch (18.42 KB, patch)
2019-09-10 04:15 PDT, youenn fablet
no flags
Patch for landing (18.19 KB, patch)
2019-09-10 08:33 PDT, youenn fablet
no flags
youenn fablet
Comment 1 2019-08-21 06:06:58 PDT
youenn fablet
Comment 2 2019-08-23 07:35:08 PDT
youenn fablet
Comment 3 2019-08-26 01:41:04 PDT
WebKit Commit Bot
Comment 4 2019-09-06 08:49:45 PDT
Comment on attachment 377235 [details] Patch Clearing flags on attachment: 377235 Committed r249574: <https://trac.webkit.org/changeset/249574>
WebKit Commit Bot
Comment 5 2019-09-06 08:49:46 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2019-09-06 08:50:20 PDT
Radar WebKit Bug Importer
Comment 7 2019-09-06 08:50:21 PDT
Alex Christensen
Comment 8 2019-09-07 14:52:53 PDT
Comment on attachment 377235 [details] Patch This seems to have made the test fast/mediastream/media-stream-track-source-failure.html fail quite often, and it was passing almost always before. https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fmediastream%2Fmedia-stream-track-source-failure.html
youenn fablet
Comment 9 2019-09-09 07:57:33 PDT
(In reply to Alex Christensen from comment #8) > Comment on attachment 377235 [details] > Patch > > This seems to have made the test > fast/mediastream/media-stream-track-source-failure.html fail quite often, > and it was passing almost always before. > https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard. > html#showAllRuns=true&tests=fast%2Fmediastream%2Fmedia-stream-track-source- > failure.html Seems like a regression, I'll check that.
Ryan Haddad
Comment 10 2019-09-09 10:52:24 PDT
Reverted r249574 for reason: Caused fast/mediastream/media-stream-track-source-failure.html to become flaky. Committed r249651: <https://trac.webkit.org/changeset/249651>
youenn fablet
Comment 11 2019-09-10 04:03:02 PDT
> Caused fast/mediastream/media-stream-track-source-failure.html to become > flaky. The issue is that the media state gets updated synchronously and so no state update is seen by the test. We just need to slightly change the test.
youenn fablet
Comment 12 2019-09-10 04:15:58 PDT
WebKit Commit Bot
Comment 13 2019-09-10 07:46:26 PDT
Comment on attachment 378456 [details] Patch Rejecting attachment 378456 [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-03', 'validate-changelog', '--check-oops', '--non-interactive', 378456, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit ChangeLog entry in LayoutTests/imported/w3c/ChangeLog contains OOPS!. Full output: https://webkit-queues.webkit.org/results/13019926
youenn fablet
Comment 14 2019-09-10 08:33:00 PDT
Created attachment 378465 [details] Patch for landing
WebKit Commit Bot
Comment 15 2019-09-10 09:06:58 PDT
Comment on attachment 378465 [details] Patch for landing Clearing flags on attachment: 378465 Committed r249718: <https://trac.webkit.org/changeset/249718>
WebKit Commit Bot
Comment 16 2019-09-10 09:07:00 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.