Bug 161651 – [WebCore] Clean up script loading code in XML

• Status: RESOLVED
• Resolution: FIXED
• Component: XML

[Yusuke Suzuki]

...

[Yusuke Suzuki]

Created attachment 295022 [details]
Patch

WIP

[Yusuke Suzuki]

Created attachment 295028 [details]
Patch

[Ryosuke Niwa]

Comment on attachment 295028 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=295028&action=review

> Source/WebCore/dom/ScriptElement.cpp:167
> +    // XHTML document does not support "defer" attribute semantics.
> +    // And module tag also uses defer attribute semantics. We disable script type="module" for non HTML document.

Is this behavior spec compliant!?

[Yusuke Suzuki]

Comment on attachment 295028 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=295028&action=review

>> Source/WebCore/dom/ScriptElement.cpp:167
>> +    // And module tag also uses defer attribute semantics. We disable script type="module" for non HTML document.
> 
> Is this behavior spec compliant!?

XHTML defines "defer" attribute. But WebKit does not implement it for a long time.
Once "defer" is implemented, we can reconsider enabling modules in XHTML. But I'm not sure that's worth doing (implementing defer semantics and enabling modules in XHTML).

Since "defer" semantics is not implemented, before this patch, we just ignore the modules in XHTML document. But it causes some strange behavior.
For example, if you write <script type="module"> in document, it is ignored. But if you create script tag with `type="module"` dynamically, it works.
To make it consistent, I think disabling modules until "defer" is implemented in XHTML is better.

I'll note the above description in this comment.

[Yusuke Suzuki]

Comment on attachment 295028 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=295028&action=review

>>> Source/WebCore/dom/ScriptElement.cpp:167
>>> +    // And module tag also uses defer attribute semantics. We disable script type="module" for non HTML document.
>> 
>> Is this behavior spec compliant!?
> 
> XHTML defines "defer" attribute. But WebKit does not implement it for a long time.
> Once "defer" is implemented, we can reconsider enabling modules in XHTML. But I'm not sure that's worth doing (implementing defer semantics and enabling modules in XHTML).
> 
> Since "defer" semantics is not implemented, before this patch, we just ignore the modules in XHTML document. But it causes some strange behavior.
> For example, if you write <script type="module"> in document, it is ignored. But if you create script tag with `type="module"` dynamically, it works.
> To make it consistent, I think disabling modules until "defer" is implemented in XHTML is better.
> 
> I'll note the above description in this comment.

We have a bug for that https://bugs.webkit.org/show_bug.cgi?id=123387. I'll paste this URL.
According to that bug, blink intentionally does not support it https://bugs.chromium.org/p/chromium/issues/detail?id=611136.

[Yusuke Suzuki]

Committed r208840: <http://trac.webkit.org/changeset/208840>