<rdar://problem/28663488>
Created attachment 290895 [details] Patch for EWS Needs a proper ChangeLog, but let's test this fix on the EWS.
Attachment 290895 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WebKit2/DidRemoveFrameFromHiearchyInPageCache_Bundle.cpp:41: The parameter name "bundle" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKit2/DidRemoveFrameFromHiearchyInPageCache_Bundle.cpp:41: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 2 in 7 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 290933 [details] Patch
Comment on attachment 290933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=290933&action=review > Tools/TestWebKitAPI/Tests/WebKit2/DidRemoveFrameFromHiearchyInPageCache.cpp:91 > + WKPageLoadURL(webView.page(), adoptWK(Util::createURLForResource("many-iframes", "html")).get()); > + Util::run(&finished); > + > + // Perform a couple of loads so "simple-iframe" gets kicked out of the PageCache. There is no "simple-iframe" anywhere. I think you mean "many-iframes".
Nice test though!
Created attachment 290949 [details] Patch for landing
Comment on attachment 290949 [details] Patch for landing Clearing flags on attachment: 290949 Committed r206922: <http://trac.webkit.org/changeset/206922>
All reviewed patches have been landed. Closing bug.