-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test that changing <iframe srcdoc> processes attributes #17437
Test that changing <iframe srcdoc> processes attributes #17437
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for writing these, I'd like a slightly better name for the second test, but not going to block on that.
html/semantics/embedded-content/the-iframe-element/srcdoc_process_attributes.html
Outdated
Show resolved
Hide resolved
html/semantics/embedded-content/the-iframe-element/srcdoc_process_attributes.html
Outdated
Show resolved
Hide resolved
html/semantics/embedded-content/the-iframe-element/srcdoc_process_attributes.html
Outdated
Show resolved
Hide resolved
html/semantics/embedded-content/the-iframe-element/srcdoc_process_attributes.html
Outdated
Show resolved
Hide resolved
html/semantics/embedded-content/the-iframe-element/srcdoc_process_attributes.html
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@annevk @shvaikalesh This test asserts that a srcdoc iframe's load event is fired before returning from the srcdoc attribute modification. Where in the specification is that requirement documented? |
I missed that, that's indeed wrong. @shvaikalesh would you be willing to change the relevant |
* "Changing" => "Setting", per #17437 (comment). * "load" event should fire after "srcdoc" attribute modification, per #17437 (comment).
…c> attributes processing test, a=testonly Automatic update from web-platform-tests Fix incorrect assertion in <iframe srcdoc> attributes processing test * "Changing" => "Setting", per web-platform-tests/wpt#17437 (comment). * "load" event should fire after "srcdoc" attribute modification, per web-platform-tests/wpt#17437 (comment). -- wpt-commits: 08f5e4f43dce6826d9a045eff8d85327c95ca2f3 wpt-pr: 20084
…c> attributes processing test, a=testonly Automatic update from web-platform-tests Fix incorrect assertion in <iframe srcdoc> attributes processing test * "Changing" => "Setting", per web-platform-tests/wpt#17437 (comment). * "load" event should fire after "srcdoc" attribute modification, per web-platform-tests/wpt#17437 (comment). -- wpt-commits: 08f5e4f43dce6826d9a045eff8d85327c95ca2f3 wpt-pr: 20084
…c> attributes processing test, a=testonly Automatic update from web-platform-tests Fix incorrect assertion in <iframe srcdoc> attributes processing test * "Changing" => "Setting", per web-platform-tests/wpt#17437 (comment). * "load" event should fire after "srcdoc" attribute modification, per web-platform-tests/wpt#17437 (comment). -- wpt-commits: 08f5e4f43dce6826d9a045eff8d85327c95ca2f3 wpt-pr: 20084 UltraBlame original commit: 1b5b5472a966c53402fc44b09c6e47f084691eab
…c> attributes processing test, a=testonly Automatic update from web-platform-tests Fix incorrect assertion in <iframe srcdoc> attributes processing test * "Changing" => "Setting", per web-platform-tests/wpt#17437 (comment). * "load" event should fire after "srcdoc" attribute modification, per web-platform-tests/wpt#17437 (comment). -- wpt-commits: 08f5e4f43dce6826d9a045eff8d85327c95ca2f3 wpt-pr: 20084 UltraBlame original commit: 1b5b5472a966c53402fc44b09c6e47f084691eab
…c> attributes processing test, a=testonly Automatic update from web-platform-tests Fix incorrect assertion in <iframe srcdoc> attributes processing test * "Changing" => "Setting", per web-platform-tests/wpt#17437 (comment). * "load" event should fire after "srcdoc" attribute modification, per web-platform-tests/wpt#17437 (comment). -- wpt-commits: 08f5e4f43dce6826d9a045eff8d85327c95ca2f3 wpt-pr: 20084 UltraBlame original commit: 1b5b5472a966c53402fc44b09c6e47f084691eab
Blink bug: srcdoc removal does not process iframe attributes.
WebKit bug:
<iframe>
attributes should be processed on "srcdoc" attribute removal.