Bug 146657 – [WK2] UIProcess/WebProcess shutdown logic may have race condition

• Status: NEW
• Component: WebKit2

[Brent Fulgham]

While fixing Bug 146554, I found that I had to update WebProcess::didClose to forcibly stop all media playback.

This was because the standard handling for WebPage::Close was getting missed due to the IPC connection between UIProcess and WebProcess getting terminated before the WebPage::Close message was received.

This should not be needed, and should be investigated.

[Radar WebKit Bug Importer]

<rdar://problem/21693748>

[Brent Fulgham]

Created attachment 256250 [details]
Logging of "good" and "bad" disconnection.

Attaching some logging I did.
(1) "Good" is the case where the WebPage::Close message is properly received by the WebProcess, and everything shuts down cleanly. This is the typical case.

(2) "Bad" is the case where the IPC connection between UIProcess and WebProcess is severed before WebPage::Close is received by the WebProcess.

[Brent Fulgham]

Created attachment 256256 [details]
Logging additions to explore problem. NOT A FIX.

[WebKit Commit Bot]

Attachment 256256 [details] did not pass style-queue:


ERROR: Source/WebKit2/Shared/ChildProcess.cpp:143:  Alphabetical sorting problem.  [build/include_order] [4]
ERROR: Source/WebKit2/Platform/IPC/Connection.cpp:710:  "syslog.h" already included at Source/WebKit2/Platform/IPC/Connection.cpp:347  [build/include] [4]
Total errors found: 2 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.