New issue
Advanced search Search tips

Issue 721560 link

Starred by 2 users
Status: Fixed
Owner:
kouhei@chromium.org
Closed: Jun 2017
Cc:
kouhei@chromium.org
hirosh...@chromium.org
neis@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug

Blocking:
issue 594639



Sign in to add a comment

ES Modules: double 404 script source crashes renderer

Project Member Reported by paulir...@chromium.org, May 11 2017 
repro:

in html:
   <script src="app.js" type="module"></script>

in app.js:
  import 'thing/404.js'
  import './other404.js'
  console.log('hi');


crash bug: https://bugs.chromium.org/p/chromium/issues/detail?id=717477
my crash: https://crash.corp.google.com/browse?q=reportid=%27290d6e88d0000000%27

Comment 1 by paulir...@chromium.org, May 12 2017

Cc: kouhei@chromium.org
Status: Untriaged (was: Available)

Comment 2 by neis@chromium.org, May 12 2017

Blocking: 594639
Cc: hirosh...@chromium.org
Looks like the one in #1 of https://bugs.chromium.org/p/chromium/issues/detail?id=718442

Comment 3 by tkent@chromium.org, May 26 2017

Owner: kouhei@chromium.org
Status: Assigned (was: Untriaged)

Comment 4 by kouhei@chromium.org, May 29 2017

Labels: -Pri-2 Pri-1
Bumping to P1 as this would block shipping.

Comment 5 by kouhei@chromium.org, May 29 2017 

help appreciated: LayoutTest w/ TestExpectation fail.
- 404.js: Just reference 404.js w/o placing file 404.js
- delayed load: I'm going to introduce delayed-modulescript.py soon in https://chromium-review.googlesource.com/c/517684/

Comment 6 by neis@chromium.org, May 29 2017 

I added this as a WPT test in https://codereview.chromium.org/2845983002/, file specifier-error.html.

Comment 7 by kouhei@chromium.org, Jun 14 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment