Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(454)

Issue 2566103002: [debug-wrappers] remove mirror tests. (Closed)

Created:
4 years ago by Yang
Modified:
4 years ago
Reviewers:
jgruber
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[debug-wrappers] remove mirror tests. Debug mirrors will no longer be supported in the near future. It will now only be tested by being used by the v8-inspector. R=jgruber@chromium.org BUG=v8:5530 Committed: https://crrev.com/825dd8a90484f510743e557062d488c27912ff7c Cr-Commit-Position: refs/heads/master@{#41686}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2012 lines) Patch
D test/mjsunit/debug-toggle-mirror-cache.js View 1 chunk +0 lines, -40 lines 0 comments Download
D test/mjsunit/es6/generators-mirror.js View 1 chunk +0 lines, -112 lines 0 comments Download
D test/mjsunit/es6/mirror-collections.js View 1 chunk +0 lines, -165 lines 0 comments Download
D test/mjsunit/es6/mirror-iterators.js View 1 chunk +0 lines, -103 lines 0 comments Download
D test/mjsunit/es6/mirror-promises.js View 1 chunk +0 lines, -90 lines 0 comments Download
D test/mjsunit/es6/mirror-symbols.js View 1 chunk +0 lines, -38 lines 0 comments Download
D test/mjsunit/harmony/mirror-async-function.js View 1 chunk +0 lines, -76 lines 0 comments Download
D test/mjsunit/harmony/mirror-async-function-promise.js View 1 chunk +0 lines, -93 lines 0 comments Download
D test/mjsunit/mirror-array.js View 1 chunk +0 lines, -138 lines 0 comments Download
D test/mjsunit/mirror-boolean.js View 1 chunk +0 lines, -59 lines 0 comments Download
D test/mjsunit/mirror-date.js View 1 chunk +0 lines, -77 lines 0 comments Download
D test/mjsunit/mirror-error.js View 1 chunk +0 lines, -94 lines 0 comments Download
D test/mjsunit/mirror-function.js View 1 chunk +0 lines, -91 lines 0 comments Download
D test/mjsunit/mirror-null.js View 1 chunk +0 lines, -50 lines 0 comments Download
D test/mjsunit/mirror-number.js View 1 chunk +0 lines, -77 lines 0 comments Download
D test/mjsunit/mirror-object.js View 1 chunk +0 lines, -291 lines 0 comments Download
D test/mjsunit/mirror-regexp.js View 1 chunk +0 lines, -101 lines 0 comments Download
D test/mjsunit/mirror-script.js View 1 chunk +0 lines, -88 lines 0 comments Download
D test/mjsunit/mirror-string.js View 1 chunk +0 lines, -89 lines 0 comments Download
D test/mjsunit/mirror-undefined.js View 1 chunk +0 lines, -50 lines 0 comments Download
D test/mjsunit/mirror-unresolved-function.js View 1 chunk +0 lines, -81 lines 0 comments Download
D test/mjsunit/regress/regress-crbug-629996.js View 1 chunk +0 lines, -9 lines 0 comments Download

Messages

Total messages: 13 (8 generated)
Yang
4 years ago (2016-12-12 10:47:11 UTC) #1
jgruber
lgtm
4 years ago (2016-12-12 13:13:27 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2566103002/1
4 years ago (2016-12-14 06:11:55 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-14 06:36:08 UTC) #11
commit-bot: I haz the power
4 years ago (2016-12-14 06:37:16 UTC) #13
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/825dd8a90484f510743e557062d488c27912ff7c
Cr-Commit-Position: refs/heads/master@{#41686}

Powered by Google App Engine
This is Rietveld 408576698