Bug 196513

Summary: Web Inspector: Remote Inspector indicate callback should always happen on the main thread
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, benjamin, cdumez, cmarcelo, commit-queue, dbates, ews-watchlist, hi, inspector-bugzilla-changes, joepeck, keith_miller, mark.lam, msaboff, saam, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix
hi: review+
[PATCH] For Bots none

Joseph Pecoraro
Reported 2019-04-02 16:35:14 PDT
Remote Inspector indicate callback should always happen on the main thread Indicator show/hide affect the view hierarchy. When there was a WebThread this would happen on the WebThread, but it would be better to happen on the MainThread with the WebThreadLock.
Attachments
[PATCH] Proposed Fix (6.12 KB, patch)
2019-04-02 16:37 PDT, Joseph Pecoraro
hi: review+
[PATCH] For Bots (6.12 KB, patch)
2019-04-02 16:43 PDT, Joseph Pecoraro
no flags
Joseph Pecoraro
Comment 1 2019-04-02 16:35:24 PDT
Joseph Pecoraro
Comment 2 2019-04-02 16:37:09 PDT
Created attachment 366547 [details] [PATCH] Proposed Fix
Devin Rousso
Comment 3 2019-04-02 16:41:49 PDT
Comment on attachment 366547 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=366547&action=review r=me, looks good to me (especially because I was there when he was working on it) > Source/WebCore/ChangeLog:11 > + When we have a WebThread, don't just run on the WebThread run NIT: there should be a semicolon between "WebThread" and "run".
Joseph Pecoraro
Comment 4 2019-04-02 16:43:55 PDT
Created attachment 366550 [details] [PATCH] For Bots
WebKit Commit Bot
Comment 5 2019-04-03 05:42:44 PDT
Comment on attachment 366550 [details] [PATCH] For Bots Clearing flags on attachment: 366550 Committed r243795: <https://trac.webkit.org/changeset/243795>
Blaze Burg
Comment 6 2019-04-03 11:52:10 PDT
Comment on attachment 366547 [details] [PATCH] Proposed Fix LGTM
Note You need to log in before you can comment on or make changes to this bug.