Michael Achenbach ee1db48cc3 Revert "[inspector] moved var initialization break location before init expression" ... This reverts commit 7a9cc70492. Reason for revert: Changes layout tests: https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/15882 This is about: inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html Original change's description: > [inspector] moved var initialization break location before init expression > > This CL improves break locations for expressions like 'var a = <expr>'. Without CL we use <expr> position as break location for initialization statement, with this CL we use position of first character after '=' as position. > Benefits (see test for details): > - only one break in expressions which includes mix of property lookup and calls, e.g. var p = Promise.resolve().then(x => x * 2), > - removed redundant break location for expressions like: let { x, y } = { x: 1, y: 2}. > > Bug: v8:5909 > Change-Id: I039d911903a2826c9859710a63ab0462c992e11b > Reviewed-on: https://chromium-review.googlesource.com/513926 > Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> > Reviewed-by: Marja Hölttä <marja@chromium.org> > Reviewed-by: Dmitry Gozman <dgozman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#45530} TBR=dgozman@chromium.org,marja@chromium.org,kozyatinskiy@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:5909 Change-Id: Ibf84401e8050d3c84db219d983de2c6bba0f697f Reviewed-on: https://chromium-review.googlesource.com/518102 Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#45547}		2017-05-29 07:03:23 +00:00
..
bugs/harmony	[debug-wrapper] migrate some scope related tests	2016-12-14 07:20:33 +00:00
debug	Revert "[inspector] moved var initialization break location before init expression"	2017-05-29 07:03:23 +00:00
protocol	[debugger] basic test infrastructure for new debugger test api.	2016-10-21 06:38:05 +00:00
regress	[interpreter] Avoid redundant {PopContext} instructions.	2017-05-24 11:58:47 +00:00
wrapper	[debugger] Various break-related functionality in test wrapper	2016-10-28 08:18:34 +00:00
debugger.gyp	[debugger] basic test infrastructure for new debugger test api.	2016-10-21 06:38:05 +00:00
debugger.isolate	[build] Remove remaining disable-inspector options in GN	2017-04-05 17:59:07 +00:00
debugger.status	[test] Skip debugger tests with old pipeline and mac asan	2017-04-28 11:12:36 +00:00
test-api.js	[debug] Handle OOM events in debugger tests	2017-02-16 11:48:22 +00:00
testcfg.py	Revert of [test] add --no-harness option to debugger tests. (patchset #1 id:1 of https://codereview.chromium.org/2831083003/ )	2017-05-08 15:06:42 +00:00