Bug 168009 - An iframe without src attribute is never loaded inside the HTML parser
Summary: An iframe without src attribute is never loaded inside the HTML parser
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-08 11:50 PST by Ryosuke Niwa
Modified: 2022-05-31 10:00 PDT (History)
4 users (show)

See Also:

Attachments
Demo (224 bytes, text/html)
2017-02-08 11:53 PST, Ryosuke Niwa 		no flags Details
View All 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 Ryosuke Niwa 2017-02-08 11:50:43 PST 
<body>
<div>
<b id="b"><p><iframe id="f"></iframe><script> // p inside b triggers the adoption agency algorithm.

alert(f.contentWindow);

window.addEventListener('load', () => {
    alert(f.contentWindow);
})

</script></b>

Both of them say "undefined" but shouldn't. This is a major regression.
Comment 1 Ryosuke Niwa 2017-02-08 11:51:48 PST 
Already regressed in r207475.
Comment 2 Ryosuke Niwa 2017-02-08 11:52:22 PST 
Regressed in r203347.
Comment 3 Ryosuke Niwa 2017-02-08 11:53:09 PST 
Will test further back in the history once I'm in the office.
Comment 4 Ryosuke Niwa 2017-02-08 11:53:37 PST 
Created attachment 300933 [details]
Demo
Comment 5 Ryosuke Niwa 2017-02-08 14:19:02 PST 
Wow, this bug exists in Safari 9!
Comment 6 Ryosuke Niwa 2017-02-08 14:33:59 PST 
Huh, maybe this never worked in Safari. I can even reproduce it in Safari 7.0.3
Comment 7 Ahmad Saleem 2022-05-30 11:26:00 PDT 
This seems to be fixed in Safari 15.5 on macOS 12.4 and consistent with other browsers like Chrome Canary 104 and Firefox Nightly 102.

Attached test case (demo) shows two dialog with [object Window] and then shows frame on the webpage for all browsers.

Since this seems to be fixed and consistent across all browsers. It should be marked as "Resolved Fixed" or "Resolved Invalid". Thanks!
Comment 8 Alexey Proskuryakov 2022-05-31 10:00:14 PDT 
Thank you for checking! We use "configuration changed" resolution when a bug goes away without a specific fix that we can point to.