DevTools: [esmodules] Support "Script First Statement" breakpoint
Reported by
l446240525@gmail.com,
May 12 2017
|
||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3096.0 Safari/537.36 Steps to reproduce the problem: . What is the expected behavior? What went wrong? . Did this work before? No Chrome version: 60.0.3096.0 Channel: canary OS Version: OS X 10.10.4 Flash Version:
,
May 12 2017
@l446240525 -- Thank You for the report. Could you please provide us steps to reproduce which would help us in triaging further. Thanks in Advance.
,
May 18 2017
Able to confirm that with the "Experimental web platform features" flag on Mac 60.0.3103.0, the script first statement breakpoint does not seem to apply to modules.
,
Jun 5 2017
,
Jun 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1b0342b55e0e32ab1f4775284098b9cc243e878a commit 1b0342b55e0e32ab1f4775284098b9cc243e878a Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org> Date: Mon Jun 05 17:57:01 2017 [DevTools] "Script First Statement" breakpoint for modules Added missing instrumentation for module evaluation. R=hiroshige@chromium.org,kouhei@chromium.org Bug: chromium:721641 Change-Id: I6647c35517bbbbf802afb3563f4c2a3527a07556 Reviewed-on: https://chromium-review.googlesource.com/524162 Reviewed-by: Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Cr-Commit-Position: refs/heads/master@{#477021} [add] https://crrev.com/1b0342b55e0e32ab1f4775284098b9cc243e878a/third_party/WebKit/LayoutTests/http/tests/inspector/sources/event-listener-breakpoints-script-fst-stmt-for-module-expected.txt [add] https://crrev.com/1b0342b55e0e32ab1f4775284098b9cc243e878a/third_party/WebKit/LayoutTests/http/tests/inspector/sources/event-listener-breakpoints-script-fst-stmt-for-module.html [modify] https://crrev.com/1b0342b55e0e32ab1f4775284098b9cc243e878a/third_party/WebKit/Source/bindings/core/v8/ScriptModule.cpp
,
Jun 5 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by ajha@chromium.org
, May 12 2017