• Benedikt Meurer's avatar
    [inspector] Properly filter by column number for inline scripts. · 81b91591
    Benedikt Meurer authored
    Previously `setBreakpointByUrl` and friends would only filter based on
    line number to find matching scripts. But that didn't work when there
    were multiple scripts in the same line (i.e. minified HTML), and we'd
    end up setting multiple breakpoints in different inline scripts, looking
    for the next possible break location in each of them individually.
    
    Fixed: chromium:1183664
    Also-By: pfaffe@chromium.org, kimanh@chromium.org
    Change-Id: I957811d30aa71609a38da75f33a24c0f720116f6
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2749155
    Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Kim-Anh Tran <kimanh@chromium.org>
    Auto-Submit: Benedikt Meurer <bmeurer@chromium.org>
    Reviewed-by: 's avatarKim-Anh Tran <kimanh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#73332}
    81b91591
regress-crbug-1183664.js 1.37 KB