DevTools: [regression] defining "function Map() ..." breaks developer tools
Reported by
simon.sa...@gmail.com,
Aug 10 2016
|
||||
Issue description
UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2824.0 Safari/537.36
Steps to reproduce the problem:
This JavaScript causes fatal problems:
function Map(keytype, valtype) {
if (keytype === undefined || keytype === null) {
} else {
console.log('some error');
}
}
console.log('test');
The console no longer works afterwards, and the 'some error' console statement is hit hundreds of times, even though no user code ever calls Map().
Add the above code to an HTML page and open the console to see. Confirmed in Windows 10 on dev channel and on 54.0.2825.0 canary (64-bit)
Attached is a reproducible HTML page (it really just contains the above code) and the results.
What is the expected behavior?
What went wrong?
After this happens, in the library I'm using, typing anything at all into the console is broken (never returns)
Did this work before? Yes This has occasionally broken before, not sure which versions of chrome
Chrome version: 54.0.2824.0 Channel: dev
OS Version: 10.0
Flash Version: Shockwave Flash 22.0 r0
,
Aug 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6753edc190cc71b16f0f930848a7ff8825457152 commit 6753edc190cc71b16f0f930848a7ff8825457152 Author: kozyatinskiy <kozyatinskiy@chromium.org> Date: Wed Aug 10 18:51:53 2016 [DevTools] Remove Map usage from InjectedScriptSource.js .. and fixed injected script validation script. BUG= 636409 R=dgozman@chromium.org,pfeldman@chromium.org Review-Url: https://codereview.chromium.org/2235703003 Cr-Commit-Position: refs/heads/master@{#411109} [modify] https://crrev.com/6753edc190cc71b16f0f930848a7ff8825457152/third_party/WebKit/Source/devtools/scripts/check_injected_script_source.py [modify] https://crrev.com/6753edc190cc71b16f0f930848a7ff8825457152/third_party/WebKit/Source/platform/v8_inspector/InjectedScriptSource.js
,
Aug 10 2016
Thanks for the speedy investigation!
,
Aug 10 2016
,
Aug 11 2016
[Automated comment] We don't branch for M54 for another 13 days, was this request meant for M53?
,
Aug 11 2016
,
Aug 30 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by pfeldman@chromium.org
, Aug 10 2016Labels: -Pri-2 Pri-0
Owner: kozyatinskiy@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: DevTools: [regression] defining "function Map() ..." breaks developer tools (was: defining "function Map() ..." breaks developer tools)