-
Mike Wittman authored
This reverts commit c33a38d7. Reason for revert: causing multiple test failures https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20Tests%20%281%29/85366 * media/controls-after-reload.html * media/controls-strict.html * media/controls-styling-strict.html * media/controls-styling.html * media/controls-without-preload.html * media/controls/lazy-loaded-style.html * media/controls/paint-controls-webkit-appearance-none-custom-bg.html * media/controls/paint-controls-webkit-appearance-none.html * media/video-controls-rendering.html * media/video-display-toggle.html * media/video-no-audio.html * virtual/new-remote-playback-pipeline/media/controls/lazy-loaded-style.html * virtual/new-remote-playback-pipeline/media/controls/paint-controls-webkit-appearance-none-custom-bg.html * virtual/new-remote-playback-pipeline/media/controls/paint-controls-webkit-appearance-none.html Original change's description: > Re-baseline tests after --enable-display-compositor-pixel-dump > > This CL enabled display compositor pixel dumps by default, which caused > this group of tests to fail due to either a) minor antialiasing artifacts, > or b) known differences in output due to being captured from the browser > side. > > https://chromium-review.googlesource.com/c/chromium/src/+/1213864 > > These bugs should be resolved with this CL: > - crbug.com/881040: Media controls now contain an overflow menu. > - crbug.com/667551: A bunch of tests are listed under this bug, and just > require rebaselining to fix non-material single-pixel > antialiasing failures. > - crbug.com/895556: These tests double their background size when the flag > is enabled. These were already fixed by another CL, > so they are NOT being rebaselined here, just removed > from TestExpectations. > > > Bug: 895912, 881040, 667551, 895556 > Change-Id: I14a1065df604151ed06f2a6be4da7eaa53968705 > Reviewed-on: https://chromium-review.googlesource.com/c/1303179 > Commit-Queue: Mason Freed <masonfreed@chromium.org> > Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> > Cr-Commit-Position: refs/heads/master@{#605004} TBR=chrishtr@chromium.org,mlamouri@chromium.org,masonfreed@chromium.org Change-Id: Ie9ec56c80e8af1a729479ba28b942913c71f555c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 895912, 881040, 667551, 895556 Reviewed-on: https://chromium-review.googlesource.com/c/1315942Reviewed-by:
Mike Wittman <wittman@chromium.org> Commit-Queue: Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#605061}
cf143dab