[ES6 modules] Add WPT for loading cyclic module graph w/ slow imports

nyaxt · chromium-wpt-export-bot · commit 996343f64a82 · 2017-06-14T15:56:05.000-07:00
This CL adds a layout test demonstrating the currently broken cycle handling, which currently fails on ToT. If a cyclic module graph node contained other edges that are not part of the cycle, the graph does not load reliably. (To be more specific, the graph does not load unless the other edges complete loading before we instantiate the cycle nodes). We expect to have the root cause fixed in the spec change PR: whatwg/html#2674 Bug: 594639 Change-Id: I3cf8a640a73083ad612c96fc4bbd0f01e00f6e46 Reviewed-on: https://chromium-review.googlesource.com/535413 Cr-Commit-Position: refs/heads/master@{#479534} WPT-Export-Revision: 1d8720afaeeb738690321b52bc88960ecb3b36f3
diff --git a/html/semantics/scripting-1/the-script-element/module/slow-cycle.html b/html/semantics/scripting-1/the-script-element/module/slow-cycle.html
@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<title>Cyclic graph with slow imports</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script type="module">
+import { loaded } from "./slow-module-graph-a.js";
+
+test(() => {
+  assert_true(loaded);
+}, "module graph with cycles load even if part of the graph loads slow");
+</script>
diff --git a/html/semantics/scripting-1/the-script-element/module/slow-module-graph-a.js b/html/semantics/scripting-1/the-script-element/module/slow-module-graph-a.js
@@ -0,0 +1,3 @@
+import "./slow-module-graph-b.js";
+import "./resources/delayed-modulescript.py"
+export let loaded = true;
diff --git a/html/semantics/scripting-1/the-script-element/module/slow-module-graph-b.js b/html/semantics/scripting-1/the-script-element/module/slow-module-graph-b.js
@@ -0,0 +1 @@
+import "./slow-module-graph-a.js";

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+import "./slow-module-graph-b.js";`
	`2`	`+import "./resources/delayed-modulescript.py"`
	`3`	`+export let loaded = true;`