Upgrade blink side ReadPixels size validation to consider ES3 pack parameters.

Upgrade blink side ReadPixels size validation to consider ES3 pack parameters.

Note that this CL didn't do the upgrade for unpack, although the
foundation is there.

BUG=563714
TEST=conformance2/reading/read-pixels-pack-parameters.html
R=kbr@chromium.org,bajones@chromium.org

Committed: https://crrev.com/ff942f12ced6fea94ec108035acfb15a048fe204
Cr-Commit-Position: refs/heads/master@{#368477}

[Zhenyao Mo, 2016-01-08 21:29:26]

kbr,bajones: PTAL

[Zhenyao Mo, 2016-01-08 21:29:31]

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

[commit-bot: I haz the power, 2016-01-08 21:30:29]

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

[Zhenyao Mo, 2016-01-08 21:30:31]

On 2016/01/08 21:29:26, Zhenyao Mo wrote:
> kbr,bajones: PTAL

The calculation is similar to the command buffer side.

[commit-bot: I haz the power, 2016-01-08 21:34:37]

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

[commit-bot: I haz the power, 2016-01-08 21:34:38]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Zhenyao Mo, 2016-01-08 21:42:28]

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

[commit-bot: I haz the power, 2016-01-08 21:43:45]

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

[Ken Russell (switch to Gerrit), 2016-01-08 22:13:30]

Very nice. LGTM

[commit-bot: I haz the power, 2016-01-08 23:51:28]

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

[commit-bot: I haz the power, 2016-01-08 23:51:28]

Dry run: This issue passed the CQ dry run.

[Zhenyao Mo, 2016-01-09 00:13:38]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2016-01-09 00:15:41]

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

[commit-bot: I haz the power, 2016-01-09 00:40:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-09 00:42:18]

Description was changed from

==========
Upgrade blink side ReadPixels size validation to consider ES3 pack parameters.

Note that this CL didn't do the upgrade for unpack, although the
foundation is there.

BUG=563714
TEST=conformance2/reading/read-pixels-pack-parameters.html
R=kbr@chromium.org,bajones@chromium.org
==========

to

==========
Upgrade blink side ReadPixels size validation to consider ES3 pack parameters.

Note that this CL didn't do the upgrade for unpack, although the
foundation is there.

BUG=563714
TEST=conformance2/reading/read-pixels-pack-parameters.html
R=kbr@chromium.org,bajones@chromium.org

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

[commit-bot: I haz the power, 2016-01-09 00:42:19]

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