New issue
Advanced search Search tips

Issue 655701 link

Starred by 1 user
Status: Fixed
Owner:
kozy@chromium.org
Closed: Dec 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Dynamically added scripts don't always show in dev tools

Reported by a...@scirra.com, Oct 13 2016 
UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2887.0 Safari/537.36

Steps to reproduce the problem:
1. Visit: https://www.scirra.com/labs/bugs/preloadscript/ (this was made for a different bug report but also shows the issue)
2. Open dev tools
3. Go to 'Sources' tab
4. View which source files are listed

What is the expected behavior?
preloaded.js should be listed.

What went wrong?
preloaded.js is not listed.

If you press 'reload' with dev tools open, it then appears.

Did this work before? N/A 

Chrome version: 56.0.2887.0  Channel: canary
OS Version: 10.0
Flash Version: Shockwave Flash 23.0 r0

Comment 1 by alph@chromium.org, Oct 13 2016

Owner: lushnikov@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 2 by lushnikov@chromium.org, Oct 13 2016

Owner: kozyatinskiy@chromium.org
This is probably because the script was collected by the V8.
Over to @kozyatinskiy who can confirm this

Comment 3 by kozyatinskiy@chromium.org, Oct 16 2017

Owner: kozy@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 6 2017 

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

commit dc7f28097d16476c2ee573d540e47794a5dde60a
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Wed Dec 06 19:01:41 2017

[DevTools] prepare tests before V8 roll

With one of the next roll V8 will stop running gc before reporting
scripts.

R=alph@chromium.org

Bug:  chromium:655701 
Change-Id: I58b86bb151420c0d848d3f1ccd4deb1d7890beb9
Reviewed-on: https://chromium-review.googlesource.com/806799
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Reviewed-by: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522148}
[modify] https://crrev.com/dc7f28097d16476c2ee573d540e47794a5dde60a/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/dc7f28097d16476c2ee573d540e47794a5dde60a/third_party/WebKit/LayoutTests/http/tests/devtools/bindings/suspendtarget-bindings-expected.txt
[modify] https://crrev.com/dc7f28097d16476c2ee573d540e47794a5dde60a/third_party/WebKit/LayoutTests/http/tests/devtools/bindings/suspendtarget-bindings.js
[modify] https://crrev.com/dc7f28097d16476c2ee573d540e47794a5dde60a/third_party/WebKit/LayoutTests/http/tests/devtools/bindings/suspendtarget-navigator.js
[modify] https://crrev.com/dc7f28097d16476c2ee573d540e47794a5dde60a/third_party/WebKit/LayoutTests/http/tests/devtools/startup/console-exception-source-url-expected.txt
[modify] https://crrev.com/dc7f28097d16476c2ee573d540e47794a5dde60a/third_party/WebKit/LayoutTests/http/tests/devtools/startup/console-stack-overflow-source-url-expected.txt
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 6 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed

commit 27cff23c4d7b42e43a45d2dc9e4afc77485a0aed
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Wed Dec 06 20:02:22 2017

[inspector] avoid running gc before reporting scripts

It will help us to preserve some scripts for user.

R=alph@chromium.org
TBR=jgruber@chromium.org

Bug:  chromium:655701 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
Change-Id: I6d42434148c2d9eb41c3a2af906e8c14ccf8d9a9
Reviewed-on: https://chromium-review.googlesource.com/806741
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Reviewed-by: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#49913}
[modify] https://crrev.com/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed/src/api.cc
[modify] https://crrev.com/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed/test/debugger/debug/regress/regress-1853.js
[modify] https://crrev.com/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed/test/debugger/debug/regress/regress-crbug-481896.js
[modify] https://crrev.com/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed/test/inspector/debugger/get-possible-breakpoints-master-expected.txt
[modify] https://crrev.com/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed/test/inspector/debugger/return-break-locations-expected.txt
[modify] https://crrev.com/27cff23c4d7b42e43a45d2dc9e4afc77485a0aed/test/inspector/debugger/wasm-scripts-expected.txt
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 7 2017 

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

commit 40a824f78102816516fc1789988ebfafcaecc22a
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Thu Dec 07 15:58:09 2017

[DevTools] reenable tests after v8 roll

TBR=dgozman@chromium.org

Bug:  chromium:655701 
Change-Id: I11b1d7d75cc1149cccb97fcb1852ae881c26fd90
Reviewed-on: https://chromium-review.googlesource.com/814277
Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522437}
[modify] https://crrev.com/40a824f78102816516fc1789988ebfafcaecc22a/third_party/WebKit/LayoutTests/TestExpectations

Comment 7 by kozy@chromium.org, Dec 7 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment