Bug 245814 - Disconnected frame switching to eager should not load immediately
Summary: Disconnected frame switching to eager should not load immediately
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Li
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-09-28 21:10 PDT by Frédéric Wang (:fredw)
Modified: 2022-12-19 16:57 PST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2022-09-28 21:10:27 PDT 
HTMLIFrameElement::loadDeferredFrame() should skip the openURL() step when the frame is disconnected.
Comment 1 Frédéric Wang (:fredw) 2022-09-28 21:16:43 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/4820
Comment 2 EWS 2022-09-29 02:25:03 PDT 
Committed 254991@main (de8591441940): <https://commits.webkit.org/254991@main>

Reviewed commits have been landed. Closing PR #4820 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2022-09-29 02:26:20 PDT 
<rdar://problem/100552538>
Comment 4 David Li 2022-10-27 20:14:23 PDT 
Re-opening for pull request https://github.com/apple/WebKit/pull/169
Comment 5 David Li 2022-11-01 23:27:06 PDT 
Pull request: https://github.com/apple/WebKit/pull/180
Comment 6 EWS 2022-11-02 08:02:25 PDT 
Committed 252432.750@safari-7614-branch (f55b3f37ef05): <https://commits.webkit.org/252432.750@safari-7614-branch>

Reviewed commits have been landed. Closing PR #180 and removing active labels.
Comment 7 EWS 2022-11-07 11:15:05 PST 
Committed 252432.776@safari-7614-branch (75391ceaaa91): <https://commits.webkit.org/252432.776@safari-7614-branch>

Reviewed commits have been landed. Closing PR #169 and removing active labels.
Comment 8 EWS 2022-12-19 16:57:10 PST 
Committed 258108@main (8efc8d987bd1): <https://commits.webkit.org/258108@main>

Reviewed commits have been landed. Closing PR #7869 and removing active labels.