• Michael Achenbach's avatar
    Revert "Revert "[test] Move has unexpected output to outproc."" · c1e9bc02
    Michael Achenbach authored
    This reverts commit 1685b5d2.
    
    Reason for revert: Was probably caused by infra change:
    https://crrev.com/c/845781
    
    Original change's description:
    > Revert "[test] Move has unexpected output to outproc."
    > 
    > This reverts commit 71605b3e.
    > 
    > Reason for revert: Seems to break static-initializers step:
    > https://build.chromium.org/p/client.v8/builders/V8%20Linux64/builds/22156
    > 
    > Original change's description:
    > > [test] Move has unexpected output to outproc.
    > > 
    > > Expected outcomes optimized to serialize [PASS] as None.
    > > 
    > > Keeping expected outcomes inside output processors should be
    > > optimized in the future. Few possible optimizations:
    > > - separate classes for tests that are expected to PASS - done as
    > > an example in mozilla test suite.
    > > - cache output processors inside testcase.
    > > - share output processors between copies of the same test - needs
    > > some updates to the create_variant to update outproc only if
    > > expected outcomes changed.
    > > 
    > > Bug: v8:6917
    > > Change-Id: Ie73f1dcdf17fdfc65bce27228f818b1dd1e420c9
    > > Reviewed-on: https://chromium-review.googlesource.com/843025
    > > Commit-Queue: Michael Achenbach <machenbach@chromium.org>
    > > Reviewed-by: Michael Achenbach <machenbach@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#50347}
    > 
    > TBR=machenbach@chromium.org,sergiyb@chromium.org,majeski@google.com
    > 
    > Change-Id: Ice1f3aee0a26f7f38996459d38fd6e0bd964113d
    > No-Presubmit: true
    > No-Tree-Checks: true
    > No-Try: true
    > Bug: v8:6917
    > Reviewed-on: https://chromium-review.googlesource.com/849572
    > Reviewed-by: Bill Budge <bbudge@chromium.org>
    > Commit-Queue: Bill Budge <bbudge@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#50348}
    
    TBR=bbudge@chromium.org,machenbach@chromium.org,sergiyb@chromium.org,majeski@google.com
    
    Change-Id: I7a522b6487de6e96985d223524533493eb9171f0
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:6917
    Reviewed-on: https://chromium-review.googlesource.com/848975Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
    Commit-Queue: Michael Achenbach <machenbach@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#50350}
    c1e9bc02
Name
Last commit
Last update
benchmarks Loading commit data...
build_overrides Loading commit data...
docs Loading commit data...
gni Loading commit data...
gypfiles Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.editorconfig Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...