Bug 185082

Summary: Shutdown WindowServer connections after checking in with launch services
Product: WebKit Reporter: Per Arne Vollan <pvollan>
Component: WebCore Misc.Assignee: Per Arne Vollan <pvollan>
Status: NEW    
Severity: Normal CC: bfulgham, commit-queue, ews-watchlist, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
bfulgham: review+
Patch
none
Archive of layout-test-results from ews206 for win-future none

Per Arne Vollan
Reported 2018-04-27 10:30:48 PDT
When WindowServer access is blocked in the WebContent process, we should shutdown all connections after checking in with launch services.
Attachments
Patch (1.50 KB, patch)
2018-04-27 10:37 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (1.50 KB, patch)
2018-04-27 10:38 PDT, Per Arne Vollan 		bfulgham: review+
DetailsFormatted DiffDiff
Patch (1.66 KB, patch)
2018-05-04 14:02 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from ews206 for win-future (12.73 MB, application/zip)
2018-05-04 16:12 PDT, EWS Watchlist 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Per Arne Vollan
Comment 1 2018-04-27 10:31:58 PDT
<rdar://problem/39613173>
Per Arne Vollan
Comment 2 2018-04-27 10:37:15 PDT
Created attachment 339001 [details] Patch
Per Arne Vollan
Comment 3 2018-04-27 10:38:23 PDT
Created attachment 339002 [details] Patch
Brent Fulgham
Comment 4 2018-04-27 15:44:44 PDT
Comment on attachment 339002 [details] Patch Looks great! r=me.
Per Arne Vollan
Comment 5 2018-05-04 14:02:34 PDT
Created attachment 339592 [details] Patch
Brent Fulgham
Comment 6 2018-05-04 15:20:53 PDT
Comment on attachment 339592 [details] Patch r=me if the EWS bots are happy.
EWS Watchlist
Comment 7 2018-05-04 16:12:46 PDT
Comment on attachment 339592 [details] Patch Attachment 339592 [details] did not pass win-ews (win): Output: http://webkit-queues.webkit.org/results/7569434 New failing tests: http/tests/security/canvas-remote-read-remote-video-blocked-no-crossorigin.html
EWS Watchlist
Comment 8 2018-05-04 16:12:57 PDT
Created attachment 339605 [details] Archive of layout-test-results from ews206 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews206 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Per Arne Vollan
Comment 9 2018-05-04 16:16:32 PDT
Comment on attachment 339592 [details] Patch Thanks for reviewing!
Brent Fulgham
Comment 10 2018-05-04 16:21:12 PDT
Per Arne: Did you see this test failure: http/tests/security/canvas-remote-read-remote-video-blocked-no-crossorigin.html Is that related to this change?
Per Arne Vollan
Comment 11 2018-05-04 16:23:06 PDT
(In reply to Brent Fulgham from comment #10) > Per Arne: Did you see this test failure: > > http/tests/security/canvas-remote-read-remote-video-blocked-no-crossorigin. > html > > Is that related to this change? I believe this is a flaky failure from the Windows bots.
WebKit Commit Bot
Comment 12 2018-05-04 16:42:49 PDT
Comment on attachment 339592 [details] Patch Clearing flags on attachment: 339592 Committed r231389: <https://trac.webkit.org/changeset/231389>
Note You need to log in before you can comment on or make changes to this bug.