Bug 207893

Summary: REGRESSION (256034): http/tests/media/media-stream/get-display-media-prompt.html is failing
Product: WebKit Reporter: Jacob Uphoff <jacob_uphoff>
Component: MediaAssignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric.carlson, ews-watchlist, glenn, hta, jer.noble, philipj, sergio, tommyw, webkit-bot-watchers-bugzilla, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Diff
none
Patch none

Jacob Uphoff
Reported 2020-02-18 08:48:56 PST
Created attachment 391049 [details] Diff http/tests/media/media-stream/get-display-media-prompt.html This test started to fail after the revisions in 256034 We did not catch this any sooner because there were expectations set for bug https://bugs.webkit.org/show_bug.cgi?id=206958 that were still in place after the bug there was resolved. I caught the failure when I went to check to see if the original bug was resolved and saw the test is now constantly failing. I was able to prove the regression came from 256034 by reproducing the failure with running the test in iterations with the command 'run-webkit-tests --root BUILD http/tests/media/media-stream/get-display-media-prompt.html -f --iterations 5000 --force' History: https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fmedia%2Fmedia-stream%2Fget-display-media-prompt.html Diff: See attachment
Attachments
Diff (2.91 KB, text/plain)
2020-02-18 08:48 PST, Jacob Uphoff
no flags
Patch (10.72 KB, patch)
2020-02-18 09:23 PST, youenn fablet
no flags
Radar WebKit Bug Importer
Comment 1 2020-02-18 08:49:25 PST
youenn fablet
Comment 2 2020-02-18 09:23:42 PST
WebKit Commit Bot
Comment 3 2020-02-18 10:58:09 PST
Comment on attachment 391056 [details] Patch Clearing flags on attachment: 391056 Committed r256841: <https://trac.webkit.org/changeset/256841>
WebKit Commit Bot
Comment 4 2020-02-18 10:58:10 PST
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.