Bug 243385 – srcdoc iframes fail to clear when srcdoc attribute removed

• Status: RESOLVED
• Resolution: FIXED
• Component: Frames

[Ahmad Saleem]

Hi Team,

As of right now, Safari is only browser to fail following WPT test:

https://wpt.fyi/results/html/semantics/embedded-content/the-iframe-element/srcdoc-attribute-reset.html?label=master&label=experimental&aligned&q=srcdoc-attribute-reset

Link to test - http://wpt.live/html/semantics/embedded-content/the-iframe-element/srcdoc-attribute-reset.html

While all other browser pass this test case, I tried to look into bug whether it is already reported or not but I was not able to find. Hence, I am raising this bug and just to bring to attention (if bug already exist, please mark it as duplicate).

This was fixed in Chrome in following - https://bugs.chromium.org/p/chromium/issues/detail?id=1233143

Thanks for continuous work in improving Webkit.. Keep going strong!

[Ryosuke Niwa]

lol, this is because of the following code:

void HTMLFrameElementBase::parseAttribute(const QualifiedName& name, const AtomString& value)
{
    if (name == srcdocAttr) {
        if (value.isNull()) {
            const AtomString& srcValue = attributeWithoutSynchronization(srcAttr);
            if (!srcValue.isNull())
                setLocation(stripLeadingAndTrailingHTMLSpaces(srcValue));
        } else

[Ryosuke Niwa]

Pull request: https://github.com/WebKit/WebKit/pull/2889

[Radar WebKit Bug Importer]

<rdar://problem/97885726>

[EWS]

Committed 252991@main (e540252620c9): <https://commits.webkit.org/252991@main>

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