Revert "Add viewport metadata tag to pixel video tests."

This reverts commit 6a44a630. Reason for revert: these tests are still failing on Android FYI bots. https://ci.chromium.org/p/chromium/builders/ci/Android%20FYI%20Release%20%28Nexus%205%29/19007 Original change's description: > Add viewport metadata tag to pixel video tests. > > 1. Add viewport metadata tag to the pixel video tests since they are run on > mobile devices(android). Without this tag, the webpages are not scaled > as expected according to device pixel ratio and hence was failing to match > color expectations. > > 2. This CL fixed 3 tests - Pixel_Video_MP4, > Pixel_Video_MP4_FourColors_Aspect_4x3 and Pixel_Video_MP4_FourColors_Rot_180. > > Bug: 925744 > Change-Id: I5dac913b158146f69c6b539d99fb386c3162eb8e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1698981 > Reviewed-by: Zhenyao Mo <zmo@chromium.org> > Reviewed-by: Kenneth Russell <kbr@chromium.org> > Commit-Queue: vikas soni <vikassoni@chromium.org> > Cr-Commit-Position: refs/heads/master@{#676650} TBR=zmo@chromium.org,kbr@chromium.org,vikassoni@chromium.org Change-Id: I73c25b707513ecd73e007a6942d82eb343d2fbd0 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 925744 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700162Reviewed-by: Saman Sami <samans@chromium.org> Commit-Queue: Saman Sami <samans@chromium.org> Cr-Commit-Position: refs/heads/master@{#676894}

Revert "Add viewport metadata tag to pixel video tests."
This reverts commit 6a44a630. Reason for revert: these tests are still failing on Android FYI bots. https://ci.chromium.org/p/chromium/builders/ci/Android%20FYI%20Release%20%28Nexus%205%29/19007 Original change's description: > Add viewport metadata tag to pixel video tests. > > 1. Add viewport metadata tag to the pixel video tests since they are run on > mobile devices(android). Without this tag, the webpages are not scaled > as expected according to device pixel ratio and hence was failing to match > color expectations. > > 2. This CL fixed 3 tests - Pixel_Video_MP4, > Pixel_Video_MP4_FourColors_Aspect_4x3 and Pixel_Video_MP4_FourColors_Rot_180. > > Bug: 925744 > Change-Id: I5dac913b158146f69c6b539d99fb386c3162eb8e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1698981 > Reviewed-by: Zhenyao Mo <zmo@chromium.org> > Reviewed-by: Kenneth Russell <kbr@chromium.org> > Commit-Queue: vikas soni <vikassoni@chromium.org> > Cr-Commit-Position: refs/heads/master@{#676650} TBR=zmo@chromium.org,kbr@chromium.org,vikassoni@chromium.org Change-Id: I73c25b707513ecd73e007a6942d82eb343d2fbd0 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 925744 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700162Reviewed-by: Saman Sami <samans@chromium.org> Commit-Queue: Saman Sami <samans@chromium.org> Cr-Commit-Position: refs/heads/master@{#676894}
e2fc7240 · Saman Sami · Commit Bot · 2a94be7a · e2fc7240 · e2fc7240
Commit e2fc7240 authored Jul 12, 2019 by Saman Sami Committed by Commit Bot Jul 12, 2019
4 changed files
--- a/content/test/data/gpu/pixel_video_mp4.html
+++ b/content/test/data/gpu/pixel_video_mp4.html
@@ -8,7 +8,6 @@ that the baseline images are regenerated on the next run.
 <html>
 <head>
-<meta name="viewport" content="initial-scale=1">
 <title>MP4 Video test</title>
 <style type="text/css">
 .nomargin {

--- a/content/test/data/gpu/pixel_video_mp4_four_colors_aspect_4x3.html
+++ b/content/test/data/gpu/pixel_video_mp4_four_colors_aspect_4x3.html
@@ -8,7 +8,6 @@ that the baseline images are regenerated on the next run.
 <html>
 <head>
-<meta name="viewport" content="initial-scale=1">
 <title>MP4 Video with Aspect 4x3 Test</title>
 <style type="text/css">
 .nomargin {

--- a/content/test/data/gpu/pixel_video_mp4_four_colors_rot_180.html
+++ b/content/test/data/gpu/pixel_video_mp4_four_colors_rot_180.html
@@ -8,7 +8,6 @@ that the baseline images are regenerated on the next run.
 <html>
 <head>
-<meta name="viewport" content="initial-scale=1">
 <title>MP4 Video with 180 Degree Rotation Test</title>
 <style type="text/css">
 .nomargin {

--- a/content/test/gpu/gpu_tests/test_expectations/pixel_expectations.txt
+++ b/content/test/gpu/gpu_tests/test_expectations/pixel_expectations.txt
@@ -151,6 +151,9 @@ crbug.com/911413 [ mac amd-0x679e ] Pixel_Video_MP4_FourColors_Rot_270 [ Skip ]
 crbug.com/927107 [ android ] Pixel_CSS3DBlueBox [ Failure ]
 # Fail on Nexus 5, 5X, 6, 6P, 9 and Shield TV.
+crbug.com/925744 [ android ] Pixel_Video_MP4 [ Skip ]
+crbug.com/925744 [ android ] Pixel_Video_MP4_FourColors_Aspect_4x3 [ Skip ]
+crbug.com/925744 [ android ] Pixel_Video_MP4_FourColors_Rot_180 [ Skip ]
 crbug.com/925744 [ android ] Pixel_Video_MP4_FourColors_Rot_270 [ Skip ]
 crbug.com/925744 [ android ] Pixel_Video_MP4_FourColors_Rot_90 [ Skip ]
 crbug.com/925744 [ android ] Pixel_Video_VP9 [ Skip ]