Issue 773622 in chromium: Dynamic import() should work inside set{Timeout,Interval}

4 views
Skip to first unread message

kou… via monorail

unread,
Oct 11, 2017, 5:43:22 AM10/11/17
to modul...@chromium.org
Status: Started
Owner: kou...@chromium.org
CC: modul...@chromium.org
Components: Blink>HTML>Script

New issue 773622 by kou...@chromium.org: Dynamic import() should work inside set{Timeout,Interval}
https://bugs.chromium.org/p/chromium/issues/detail?id=773622

Make Blink pass:
wpt/html/semantics/scripting-1/the-script-element/module/dynamic-import/string-compilation-base-url-classic.html
wpt/html/semantics/scripting-1/the-script-element/module/dynamic-import/string-compilation-base-url-module.html


--
You received this message because:
1. You were specifically CC'd on the issue

You may adjust your notification preferences at:
https://bugs.chromium.org/hosting/settings

Reply to this email to add a comment.

bugdro… via monorail

unread,
Oct 11, 2017, 6:51:23 AM10/11/17
to modul...@chromium.org

Comment #1 on issue 773622 by bugd...@chromium.org: Dynamic import() should work inside set{Timeout,Interval}
https://bugs.chromium.org/p/chromium/issues/detail?id=773622#c1

The following revision refers to this bug:
https://chromium.googlesource.com/chromium/src.git/+/f7bd4ce21651472a01877af861a6ec582e6f778f

commit f7bd4ce21651472a01877af861a6ec582e6f778f
Author: Kouhei Ueno <kou...@chromium.org&gt;
Date: Wed Oct 11 10:50:42 2017

[ES6 modules] Remove VirtualTestSuites for dynamic imports

They no longer need to be virtual tests since we have enabled dynamic
imports by default.

Bug: 711706, 773622
Change-Id: I4d4eb4838694ef947ab3ca795a2b6848b9d2d04f
Reviewed-on: https://chromium-review.googlesource.com/711537
Commit-Queue: Kouhei Ueno <kou...@chromium.org&gt;
Reviewed-by: Kentaro Hara <har...@chromium.org&gt;
Reviewed-by: Kinuko Yasuda <kin...@chromium.org&gt;
Cr-Commit-Position: refs/heads/master@{#507942}
[modify] https://crrev.com/f7bd4ce21651472a01877af861a6ec582e6f778f/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/f7bd4ce21651472a01877af861a6ec582e6f778f/third_party/WebKit/LayoutTests/VirtualTestSuites
[delete] https://crrev.com/961817f7c152c667729417f4aedebd8602df042e/third_party/WebKit/LayoutTests/virtual/dynamic-import/external/wpt/html/semantics/scripting-1/the-script-element/module/dynamic-import/README.txt

hiroshige via monorail

unread,
Aug 2, 2021, 12:07:47 AM8/2/21
to modul...@chromium.org
Updates:
Mergedinto: 1133238
Status: Duplicate

Comment #3 on issue 773622 by hiro...@chromium.org: Dynamic import() should work inside set{Timeout,Interval}
https://bugs.chromium.org/p/chromium/issues/detail?id=773622#c3

(No comment was entered for this change.)


--
You received this message because:
1. You were specifically CC'd on the issue

You may adjust your notification preferences at:
https://bugs.chromium.org/hosting/settings

Reply to this email to add a comment or make updates.
Reply all
Reply to author
Forward
0 new messages