Summary: | On load from back/forward cache, call checkCompleted() for ALL frames inside FrameLoader::commitProvisionalLoad() | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | Page Loading | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | achristensen, beidson, ews-watchlist, ggaren, japhet, webkit-bug-importer, youennf | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=156356 | ||||||||
Attachments: |
|
Description
Chris Dumez
2020-06-26 13:09:42 PDT
Created attachment 402893 [details]
Patch
Created attachment 402894 [details]
Patch
ping review? Comment on attachment 402894 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402894&action=review > Source/WebCore/loader/FrameLoader.cpp:-2080 > - checkCompleted(); Is there a potential change of behavior in case checkCompleted done here would have changed the list of children iframes below? > Source/WebCore/loader/FrameLoader.cpp:2082 > + for (Frame* child = m_frame.tree().firstChild(); child; child = child->tree().traverseNext(&m_frame)) auto* Committed r263662: <https://trac.webkit.org/changeset/263662> |