Bug 146657

Summary: [WK2] UIProcess/WebProcess shutdown logic may have race condition
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit2Assignee: Brent Fulgham <bfulgham>
Status: NEW    
Severity: Normal CC: ap, beidson, bfulgham, commit-queue, eric.carlson, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=168126
Attachments:
Description Flags
Logging of "good" and "bad" disconnection.
none
Logging additions to explore problem. NOT A FIX. none

Brent Fulgham
Reported 2015-07-06 15:56:04 PDT
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.
Attachments
Logging of "good" and "bad" disconnection. (20.62 KB, text/plain)
2015-07-06 16:16 PDT, Brent Fulgham 		no flags
Details
Logging additions to explore problem. NOT A FIX. (15.18 KB, patch)
2015-07-06 16:47 PDT, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2015-07-06 15:59:57 PDT
<rdar://problem/21693748>
Brent Fulgham
Comment 2 2015-07-06 16:16:45 PDT
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
Comment 3 2015-07-06 16:47:32 PDT
Created attachment 256256 [details] Logging additions to explore problem. NOT A FIX.
WebKit Commit Bot
Comment 4 2015-07-06 16:49:47 PDT
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.
Note You need to log in before you can comment on or make changes to this bug.