• Aleksey Kozyatinskiy's avatar
    [inspector] moved var initialization break location before init expression (reland) · fb6a094d
    Aleksey Kozyatinskiy authored
    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}.
     
    TBR=dgozman@chromium.org,rmcilroy@chromium.org,machenbach@chromium.org,marja@chromium.org,kozyatinskiy@chromium.org,devtools-reviews@chromium.org,v8-reviews@googlegroups.com
    # Not skipping CQ checks because original CL landed > 1 day ago.
    Bug: v8:5909
    
    Change-Id: Ie84fa79afeed09e28cf8478ba610a0cfbfdfc294
    Reviewed-on: https://chromium-review.googlesource.com/518116
    Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
    Reviewed-by: 's avatarAleksey Kozyatinskiy <kozyatinskiy@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#45598}
    fb6a094d
Name
Last commit
Last update
..
async-debug-basic.js Loading commit data...
async-debug-builtin-predictions.js Loading commit data...
async-debug-caught-exception-cases.js Loading commit data...
async-debug-caught-exception-cases0.js Loading commit data...
async-debug-caught-exception-cases1.js Loading commit data...
async-debug-caught-exception-cases2.js Loading commit data...
async-debug-caught-exception-cases3.js Loading commit data...
async-debug-caught-exception.js Loading commit data...
async-debug-step-abort-at-break.js Loading commit data...
async-debug-step-continue-at-break.js Loading commit data...
async-debug-step-in-and-out.js Loading commit data...
async-debug-step-in-out-out.js Loading commit data...
async-debug-step-in.js Loading commit data...
async-debug-step-nested.js Loading commit data...
async-debug-step-next-constant.js Loading commit data...
async-debug-step-next.js Loading commit data...
async-debug-step-out.js Loading commit data...
async-function-debug-evaluate.js Loading commit data...
async-function-debug-scopes.js Loading commit data...
debug-async-break-on-stack.js Loading commit data...
debug-async-break.js Loading commit data...
debug-async-liveedit.js Loading commit data...