WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
51352
Web process should only spin a run loop while waiting for replies to synchronous messages that invoke UI
https://bugs.webkit.org/show_bug.cgi?id=51352
Summary
Web process should only spin a run loop while waiting for replies to synchron...
Adam Roben (:aroben)
Reported
2010-12-20 13:23:25 PST
As described in
bug 51351
, having the UI process respond to synchronous messages from the web process on the main thread can result in deadlocks when windowed plugins are involved. We're going to move all synchronous message processing off the main thread by default in the UI process. But synchronous messages that result in needing to display UI (e.g., WebPageProxy::RunJavaScriptAlert) require doing so on the main thread. In those cases, the web process will need to spin a run loop while waiting for the message reply to avoid the kinds of deadlocks described in
bug 51351
.
Attachments
Add attachment
proposed patch, testcase, etc.
Adam Roben (:aroben)
Comment 1
2010-12-20 13:25:05 PST
<
rdar://problem/8790537
>
Adam Roben (:aroben)
Comment 2
2011-01-26 18:10:29 PST
As described in
bug 51351 comment 4
, we're going to spin a run loop while waiting for a reply to all synchronous messages. Once
bug 51351
is fixed we'll only spin a run loop while waiting for replies to synchronous messages that present UI in the UI process.
Adam Roben (:aroben)
Comment 3
2011-04-11 09:08:56 PDT
(In reply to
comment #2
)
> As described in
bug 51351 comment 4
, we're going to spin a run loop while waiting for a reply to all synchronous messages.
Bug 58239
now covers this strategy.
> Once
bug 51351
is fixed we'll only spin a run loop while waiting for replies to synchronous messages that present UI in the UI process.
We'll continue to use this bug to track this future improvement.
Anders Carlsson
Comment 4
2015-06-18 14:56:35 PDT
We got rid of WK2 for Windows so this no longer applies.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug