From e4de8b4626f8872a4cb1ee241b60902653567503 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?=
 <m.goleb@gmail.com>
Date: Mon, 21 Jan 2019 18:34:40 +0100
Subject: [PATCH] Manipulation: Respect script nomodule attribute in DOM
 manipulation

PR #3869 added support for `<script type="module">` & some support for
the `nomodule` attribute but with no tests for `nomodule` and with the
attribute only respected on inline scripts. This commit adds support for
source-based scripts as well. It also adds tests for `nomodule`, including
making sure legacy browsers execute such scripts as they'd natively do - that's
the whole point of `nomodule` scripts, after all.

Fixes gh-4281
Closes gh-4282
Ref gh-3871
Ref gh-3869
---
 src/manipulation.js         |  2 +-
 test/data/inner_nomodule.js |  1 +
 test/data/nomodule.js       |  1 +
 test/unit/manipulation.js   | 32 +++++++++++++++++++++++++++++++-
 4 files changed, 34 insertions(+), 2 deletions(-)
 create mode 100644 test/data/inner_nomodule.js
 create mode 100644 test/data/nomodule.js

diff --git a/src/manipulation.js b/src/manipulation.js
index a0e93f7ef..042728573 100644
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -198,7 +198,7 @@ function domManip( collection, args, callback, ignored ) {
 						if ( node.src && ( node.type || "" ).toLowerCase()  !== "module" ) {
 
 							// Optional AJAX dependency, but won't run scripts if not present
-							if ( jQuery._evalUrl ) {
+							if ( jQuery._evalUrl && !node.noModule ) {
 								jQuery._evalUrl( node.src );
 							}
 						} else {
diff --git a/test/data/inner_nomodule.js b/test/data/inner_nomodule.js
new file mode 100644
index 000000000..472f9d6b1
--- /dev/null
+++ b/test/data/inner_nomodule.js
@@ -0,0 +1 @@
+window.ok( !QUnit.moduleTypeSupported, "evaluated: inner nomodule script with src" );
diff --git a/test/data/nomodule.js b/test/data/nomodule.js
new file mode 100644
index 000000000..fd26ea4e0
--- /dev/null
+++ b/test/data/nomodule.js
@@ -0,0 +1 @@
+window.ok( !QUnit.moduleTypeSupported, "evaluated: nomodule script with src" );
diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js
index 672f397ad..7f30ddf25 100644
--- a/test/unit/manipulation.js
+++ b/test/unit/manipulation.js
@@ -1798,7 +1798,7 @@ QUnit.test( "html(Function)", function( assert ) {
 } );
 
 QUnit[
-	// Support: Edge 16-17
+	// Support: Edge 16-18+
 	// Edge sometimes doesn't execute module scripts so skip the test there.
 	( QUnit.moduleTypeSupported && !/edge\//i.test( navigator.userAgent ) ) ?
 		"test" :
@@ -1825,6 +1825,36 @@ QUnit[
 	}, 1000 );
 } );
 
+QUnit[
+	// Support: IE 9-11 only, Android 4.0-4.4 only, iOS 7-10 only
+	// `nomodule` scripts should be executed by legacy browsers only.
+	// iOS 10 supports `<script type="module">` but doesn't support the nomodule attribute
+	// so let's skip it here; sites supporting it must handle `nomodule` in a custom way anyway.
+	!/iphone os 10_/i.test( navigator.userAgent ) ?
+		"test" :
+		"skip"
+]( "html(script nomodule)", function( assert ) {
+	assert.expect( QUnit.moduleTypeSupported ? 0 : 4 );
+	var done = assert.async(),
+		$fixture = jQuery( "#qunit-fixture" );
+
+	$fixture.html(
+		[
+			"<script nomodule>ok( !QUnit.moduleTypeSupported, 'evaluated: nomodule script' );</script>",
+			"<script nomodule src='" + url( "nomodule.js" ) + "'></script>",
+			"<div>",
+				"<script nomodule>ok( !QUnit.moduleTypeSupported, 'evaluated: inner nomodule script' );</script>",
+				"<script nomodule src='" + url( "inner_nomodule.js" ) + "'></script>",
+			"</div>"
+		].join( "" )
+	);
+
+	// Allow asynchronous script execution to generate assertions
+	setTimeout( function() {
+		done();
+	}, 1000 );
+} );
+
 QUnit.test( "html(Function) with incoming value -- direct selection", function( assert ) {
 
 	assert.expect( 4 );
-- 
2.39.5