• Benedikt Meurer's avatar
    [inspector] Fix mapping between location and offset. · d821a6a3
    Benedikt Meurer authored
    We weren't really translating between location (line and column number)
    and source position (character offset) consistently, especially when it
    came to inline <script>s. There were also inconsistencies between what
    Debugger.getPossibleBreakpoints and Debugger.setBreakpointByUrl would
    do.
    
    With this CL, we are now consistently operating under the following
    assumptions:
    
    (1) For inline <scripts>s with a //@ sourceURL annotation, we assume
        that the line and column number that comes in via the protocol is
        in terms of the source text of the script.
    (2) For inline <script>s without said annotation, we assume that the
        line and column numbers are in terms of the surrounding document.
    
    This is finally aligned with how the DevTools front-end operates.
    
    Fixed: chromium:1319828
    Change-Id: I98c4ef04b34a97caf060ff4f32690b135edb6ee6
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3610622Reviewed-by: 's avatarKim-Anh Tran <kimanh@chromium.org>
    Auto-Submit: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Kim-Anh Tran <kimanh@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#80292}
    d821a6a3
Name
Last commit
Last update
..
console Loading commit data...
cpu-profiler Loading commit data...
debugger Loading commit data...
heap-profiler Loading commit data...
regress Loading commit data...
runtime Loading commit data...
sessions Loading commit data...
type-profiler Loading commit data...
BUILD.gn Loading commit data...
DEPS Loading commit data...
DIR_METADATA Loading commit data...
OWNERS Loading commit data...
frontend-channel.h Loading commit data...
inspector-test.cc Loading commit data...
inspector.status Loading commit data...
isolate-data.cc Loading commit data...
isolate-data.h Loading commit data...
json-parse-expected.txt Loading commit data...
json-parse.js Loading commit data...
print-method-not-found-expected.txt Loading commit data...
print-method-not-found.js Loading commit data...
protocol-test.js Loading commit data...
task-runner.cc Loading commit data...
task-runner.h Loading commit data...
tasks.cc Loading commit data...
tasks.h Loading commit data...
testcfg.py Loading commit data...
utils.cc Loading commit data...
utils.h Loading commit data...
wasm-inspector-test.js Loading commit data...