Unverified Commit f634f37b authored by Benjamin Coe's avatar Benjamin Coe
Browse files

deps: V8: cherry-pick deac757

Original commit message:

    [debugger] Fix code coverage for break/return inside switch-case

    Case statements have a list of statements associated with them, but are
    not blocks, and were hence not fixed-up correctly for code coverage.
    This CL also applies the fix-up to the "body" of case statements,
    in this way removing ranges reported as uncovered between the final
    break/return in a case and the next case (or end of function).

    Drive-by: Add optional pretty printing to code coverage test results.

    Change-Id: I5f4002d4e17b7253ed516d99f7c389ab2264be10
    Bug: v8:9705
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1798426


Reviewed-by: default avatarToon Verwaest <verwaest@chromium.org>
Reviewed-by: default avatarJakob Gruber <jgruber@chromium.org>
    Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#63719}

Refs: https://github.com/v8/v8/commit/deac757bc76d207845375340bcea090c774ad9d0

PR-URL: https://github.com/nodejs/node/pull/29626


Reviewed-By: default avatarRich Trott <rtrott@gmail.com>
Reviewed-By: default avatarRichard Lau <riclau@uk.ibm.com>
Reviewed-By: default avatarShelley Vohr <codebytere@gmail.com>
Reviewed-By: default avatarAnna Henningsen <anna@addaleax.net>
parent 9f830f37
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment