New issue
Advanced search Search tips

Issue 805249 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 27
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[DevTools] async await with for var of loop does not work

Project Member Reported by kozy@chromium.org, Jan 24 2018

Issue description

await Promise.resolve();
for (var a of [1,2,3]) {}
 

Comment 1 by kozy@chromium.org, Jan 24 2018

Summary: [DevTools] async await with for var of loop does not work (was: [DevTools] async await with for var of look does not work)
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 27

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

commit 653c93538b630a10837fe7e5014a10000e103e37
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Thu Sep 27 19:34:54 2018

DevTools: top level await rewrite should not change for of var decl

We can not replace it with our regular void (a = ...) here, so let's
just ignore it.

R=lushnikov@chromium.org

Bug:  chromium:805249 
Change-Id: I859ca107a15a0284a7c6a2ed0e32fe0c11d4734f
Reviewed-on: https://chromium-review.googlesource.com/1249539
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594816}
[modify] https://crrev.com/653c93538b630a10837fe7e5014a10000e103e37/third_party/WebKit/LayoutTests/http/tests/devtools/unit/preprocess-top-level-awaits-expected.txt
[modify] https://crrev.com/653c93538b630a10837fe7e5014a10000e103e37/third_party/WebKit/LayoutTests/http/tests/devtools/unit/preprocess-top-level-awaits.js
[modify] https://crrev.com/653c93538b630a10837fe7e5014a10000e103e37/third_party/blink/renderer/devtools/front_end/formatter_worker/FormatterWorker.js

Status: Fixed (was: Assigned)

Sign in to add a comment