diff options
author | Benjamin Coe <bencoe@google.com> | 2019-09-19 19:50:23 -0700 |
---|---|---|
committer | Benjamin Coe <bencoe@google.com> | 2019-09-21 10:59:35 -0700 |
commit | f634f37be0f9464ab6e5318cd88af89cd416b5a6 (patch) | |
tree | 7fece3a49b4c0bfd93363e3bc5b6c1d805d75352 /common.gypi | |
parent | 9f830f37daf55a2c903153e1681873a8cd61baee (diff) | |
download | android-node-v8-f634f37be0f9464ab6e5318cd88af89cd416b5a6.tar.gz android-node-v8-f634f37be0f9464ab6e5318cd88af89cd416b5a6.tar.bz2 android-node-v8-f634f37be0f9464ab6e5318cd88af89cd416b5a6.zip |
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: https://github.com/v8/v8/commit/deac757bc76d207845375340bcea090c774ad9d0
PR-URL: https://github.com/nodejs/node/pull/29626
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Diffstat (limited to 'common.gypi')
-rw-r--r-- | common.gypi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common.gypi b/common.gypi index ca1f853172..10b4ecfc52 100644 --- a/common.gypi +++ b/common.gypi @@ -39,7 +39,7 @@ # Reset this number to 0 on major V8 upgrades. # Increment by one for each non-official patch applied to deps/v8. - 'v8_embedder_string': '-node.12', + 'v8_embedder_string': '-node.13', ##### V8 defaults for Node.js ##### |