Summary: | New Layout Test landed flaky [Mac WK1] resize-observer/element-leak.html is a flaky failure | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Shawn Roberts <sroberts> | ||||||||||||
Component: | Tools / Tests | Assignee: | cathiechen <cathiechen> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | cathiechen, commit-queue, ews-watchlist, fred.wang, jlewis3, rniwa, rwlbuis, ryanhaddad, tsavell, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=197457 | ||||||||||||||
Attachments: |
|
Description
Shawn Roberts
2019-06-07 10:48:14 PDT
*** Bug 198667 has been marked as a duplicate of this bug. *** Maybe GC is too slow in WebKit1? Maybe we just need to skip the test in WK1. The commit-queue just saw resize-observer/element-leak.html flake (text diff) while processing attachment 371632 [details] on bug 198681. Bot: webkit-cq-02 Port: <class 'webkitpy.common.config.ports.MacPort'> Platform: Mac OS X 10.13.6 Created attachment 371636 [details]
Archive of layout-test-results from webkit-cq-02
Looks like it took very long time to construct iframes which create 1000 div and remove. I'll reduce the div number. We don't need 1000 div to confirm the GC problem. Created attachment 371647 [details]
Patch
Comment on attachment 371647 [details] Patch Attachment 371647 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/12416260 New failing tests: fast/block/float/float-with-anonymous-previous-sibling.html Created attachment 371649 [details]
Archive of layout-test-results from ews215 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews215 Port: win-future Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
(In reply to Build Bot from comment #7) > Comment on attachment 371647 [details] > Patch > > Attachment 371647 [details] did not pass win-ews (win): > Output: https://webkit-queues.webkit.org/results/12416260 > > New failing tests: > fast/block/float/float-with-anonymous-previous-sibling.html This is not related to the patch. Comment on attachment 371647 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=371647&action=review > LayoutTests/ChangeLog:9 > + We don't need 1000 element to confirm the GC problem. So in order to reduce the fake alarm. 1000 elements* There should be a comma at the end of the line. What is the "fake alarm", a timeout? > LayoutTests/resize-observer/resources/element-leak-frame.html:18 > +var container = document.createElement('p'); p's content model is "phrasing content", so it is not valid to put a div element inside. https://html.spec.whatwg.org/multipage/grouping-content.html#the-p-element You can make container a div instead. > LayoutTests/resize-observer/resources/element-leak-frame.html:23 > +document.body.appendChild(container); I don't know if this change is necessary, but it should be explained in the ChangeLog. Comment on attachment 371647 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=371647&action=review Hi Fred, Thanks for the review. >> LayoutTests/ChangeLog:9 >> + We don't need 1000 element to confirm the GC problem. So in order to reduce the fake alarm. > > 1000 elements* > There should be a comma at the end of the line. > What is the "fake alarm", a timeout? Done. Yes, it's a timeout. >> LayoutTests/resize-observer/resources/element-leak-frame.html:18 >> +var container = document.createElement('p'); > > p's content model is "phrasing content", so it is not valid to put a div element inside. > https://html.spec.whatwg.org/multipage/grouping-content.html#the-p-element > > You can make container a div instead. Done. >> LayoutTests/resize-observer/resources/element-leak-frame.html:23 >> +document.body.appendChild(container); > > I don't know if this change is necessary, but it should be explained in the ChangeLog. Done. This change could reduce the times to modify DOM tree directly. Created attachment 372449 [details]
Patch
Created attachment 372450 [details]
Patch
Comment on attachment 372450 [details] Patch Clearing flags on attachment: 372450 Committed r246584: <https://trac.webkit.org/changeset/246584> All reviewed patches have been landed. Closing bug. |