Summary: | Fix for several failures of LayoutTests in isolated tree mode. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andres Gonzalez <andresg_22> | ||||||
Component: | New Bugs | Assignee: | Andres Gonzalez <andresg_22> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | aboxhall, apinheiro, cfleizach, commit-queue, dmazzoni, ews-watchlist, jcraig, jdiggs, samuel_white, simon.fraser, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Andres Gonzalez
2020-03-05 12:26:53 PST
Created attachment 392606 [details]
Patch
Created attachment 392615 [details]
Patch
The commit-queue encountered the following flaky tests while processing attachment 392615 [details]: editing/spelling/spellcheck-input-search-crash.html bug 207995 (authors: arv@chromium.org, g.czajkowski@samsung.com, mark.lam@apple.com, and morrita@google.com) The commit-queue is continuing to process your patch. The commit-queue encountered the following flaky tests while processing attachment 392615 [details]: editing/spelling/spellcheck-paste-continuous-disabled.html bug 208016 (authors: g.czajkowski@samsung.com and mark.lam@apple.com) The commit-queue is continuing to process your patch. Comment on attachment 392615 [details] Patch Clearing flags on attachment: 392615 Committed r257948: <https://trac.webkit.org/changeset/257948> All reviewed patches have been landed. Closing bug. Comment on attachment 392615 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392615&action=review > Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.h:830 > + ScrollView* scrollView() const override; Is this called off the main thread? (In reply to Simon Fraser (smfr) from comment #8) > Comment on attachment 392615 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=392615&action=review > > > Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.h:830 > > + ScrollView* scrollView() const override; > > Is this called off the main thread? no, it is called in a lambda dispatched to the main thread with: return Accessibility::retrieveAutoreleasedValueFromMainThread<id>(...) The implementation in the IsolatedObject will assert if it is not on the main thread. |