Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: V8: cherry-pick deac757 #29626

Closed
wants to merge 1 commit into from
Closed

Commits on Sep 20, 2019

  1. 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: Toon Verwaest <verwaest@chromium.org>
        Reviewed-by: Jakob Gruber <jgruber@chromium.org>
        Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
        Cr-Commit-Position: refs/heads/master@{#63719}
    
    Refs: v8/v8@deac757
    bcoe committed Sep 20, 2019
    Copy the full SHA
    b2e7100 View commit details
    Browse the repository at this point in the history