Add <meta> tag YUV JPEG image decoding layout tests

Add <meta> tag YUV JPEG image decoding layout tests

Based on the test from CL issue 561363002, here updated to use
the current <meta> viewport tag that enables GPU rasterization
support on Chrome Android, and to include pixel results (which
are added here to prevent paint regressions, at least for this
very small set of YUV JPEG images).

BUG=598949
Committed: https://crrev.com/ba0c7e528c319dd596706d576f094e14b1991b54
Cr-Commit-Position: refs/heads/master@{#384062}

[Noel Gordon, 2016-03-30 07:23:22]

noel@chromium.org changed reviewers:
+ msarett@chromium.org

[Noel Gordon, 2016-03-30 07:24:21]

The CQ bit was checked by noel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-30 07:24:53]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1840303003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1840303003/1

[commit-bot: I haz the power, 2016-03-30 08:29:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-30 08:29:28]

Dry run: This issue passed the CQ dry run.

[msarett1, 2016-03-30 13:07:59]

LGTM

https://codereview.chromium.org/1840303003/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/images/jpeg-ycbcr-vuv-image-decoding.html
(right):

https://codereview.chromium.org/1840303003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/jpeg-ycbcr-vuv-image-decoding.html:4:
<meta name="viewport" content="width=device-width, minimum-scale=1.0">
Can you add a comment here mentioning that "viewport" enables gpu rasterization
on Android?  Is it interesting to also explain why?

[Noel Gordon, 2016-03-30 19:45:37]

The change description and the bug already covers it, no?

[Noel Gordon, 2016-03-30 19:45:44]

The CQ bit was checked by noel@chromium.org

[commit-bot: I haz the power, 2016-03-30 19:46:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1840303003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1840303003/1

[msarett, 2016-03-30 19:50:15]

On 2016/03/30 19:45:37, noel gordon wrote:
> The change description and the bug already covers it, no?

Well yes, though this information might not be immediately available to someone
reading the code.  I'll not nit though, I'm happy with this as is :).

[commit-bot: I haz the power, 2016-03-30 19:54:46]

Committed patchset #1 (id:1)

[Noel Gordon, 2016-03-30 20:01:43]

On 2016/03/30 19:50:15, msarett wrote:
> On 2016/03/30 19:45:37, noel gordon wrote:
> > The change description and the bug already covers it, no?
> 
> Well yes, though this information might not be immediately available to
someone
> reading the code.  I'll not nit though, I'm happy with this as is :).

Same for a lot of Blink code :) and why we have that rule: all patches should
come with a test, all patches should have a bug.  That way, many more details
about why we are making a change are documented, and that to help code
archaeologists (git blame etc).

[commit-bot: I haz the power, 2016-03-30 20:05:39]

Description was changed from

==========
Add <meta> tag YUV JPEG image decoding layout tests

Based on the test from CL issue 561363002, here updated to use
the current <meta> viewport tag that enables GPU rasterization
support on Chrome Android, and to include pixel results (which
are added here to prevent paint regressions, at least for this
very small set of YUV JPEG images).

BUG=598949
==========

to

==========
Add <meta> tag YUV JPEG image decoding layout tests

Based on the test from CL issue 561363002, here updated to use
the current <meta> viewport tag that enables GPU rasterization
support on Chrome Android, and to include pixel results (which
are added here to prevent paint regressions, at least for this
very small set of YUV JPEG images).

BUG=598949

Committed: https://crrev.com/ba0c7e528c319dd596706d576f094e14b1991b54
Cr-Commit-Position: refs/heads/master@{#384062}
==========

[commit-bot: I haz the power, 2016-03-30 20:05:39]

Patchset 1 (id:??) landed as
https://crrev.com/ba0c7e528c319dd596706d576f094e14b1991b54
Cr-Commit-Position: refs/heads/master@{#384062}

[Noel Gordon, 2016-03-30 20:25:58]

On 2016/03/30 19:50:15, msarett wrote:
> On 2016/03/30 19:45:37, noel gordon wrote:
> > The change description and the bug already covers it, no?
> 
> Well yes, though this information might not be immediately available to
someone
> reading the code.  I'll not nit though, I'm happy with this as is :).

Since I needed to rename the test anyway, I'll add an explicit bug reference to
the test.  Please review https://codereview.chromium.org/1836293004