Decouple GLHelper from media

Decouple GLHelper from media

We would like to share GLHelper with Mus. In order to do so, we would like
to place this file in gpu/. Thus, it cannot depend on media/.

This CL removes the dependency on media::VideoFrame by passing in the
stride and data of the Y, U, and V planes into ReadbackYUV instead of
passing in a media::VideoFrame. The callers are responsible for keeping
media::VideoFrame alive now instead of ReadbackYUV.

This CL also moves media::LetterboxYUV to the callers.

BUG=601866
Committed: https://crrev.com/59dcea480d282e78805de1c6e15ec29f4ebe7c9d
Cr-Commit-Position: refs/heads/master@{#387701}

[Fady Samuel, 2016-04-14 22:55:40]

fsamuel@chromium.org changed reviewers:
+ piman@chromium.org

[Fady Samuel, 2016-04-14 22:56:41]

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

[commit-bot: I haz the power, 2016-04-14 22:56:55]

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

[commit-bot: I haz the power, 2016-04-14 23:14:37]

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

[commit-bot: I haz the power, 2016-04-14 23:14:38]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[piman, 2016-04-14 23:20:04]

lgtm

[Fady Samuel, 2016-04-15 15:01:40]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-04-15 15:02:05]

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

[commit-bot: I haz the power, 2016-04-15 15:15:49]

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

[commit-bot: I haz the power, 2016-04-15 15:15:50]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[Fady Samuel, 2016-04-15 17:08:22]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-04-15 17:08:23]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1893473002/#ps20001
(title: "Fixed unit tests")

[commit-bot: I haz the power, 2016-04-15 17:08:49]

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

[commit-bot: I haz the power, 2016-04-15 17:34:20]

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

[commit-bot: I haz the power, 2016-04-15 17:34:21]

Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Fady Samuel, 2016-04-15 20:40:28]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-04-15 20:40:28]

The CQ bit was unchecked by fsamuel@chromium.org

[Fady Samuel, 2016-04-15 20:40:29]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1893473002/#ps40001
(title: "Fixed content_browsertests")

[Fady Samuel, 2016-04-15 20:40:29]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-04-15 20:40:53]

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

[commit-bot: I haz the power, 2016-04-15 21:27:58]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-15 21:29:37]

Description was changed from

==========
Decouple GLHelper from media

We would like to share GLHelper with Mus. In order to do so, we would like
to place this file in gpu/. Thus, it cannot depend on media/.

This CL removes the dependency on media::VideoFrame by passing in the
stride and data of the Y, U, and V planes into ReadbackYUV instead of
passing in a media::VideoFrame. The callers are responsible for keeping
media::VideoFrame alive now instead of ReadbackYUV.

This CL also moves media::LetterboxYUV to the callers.

BUG=601866
==========

to

==========
Decouple GLHelper from media

We would like to share GLHelper with Mus. In order to do so, we would like
to place this file in gpu/. Thus, it cannot depend on media/.

This CL removes the dependency on media::VideoFrame by passing in the
stride and data of the Y, U, and V planes into ReadbackYUV instead of
passing in a media::VideoFrame. The callers are responsible for keeping
media::VideoFrame alive now instead of ReadbackYUV.

This CL also moves media::LetterboxYUV to the callers.

BUG=601866

Committed: https://crrev.com/59dcea480d282e78805de1c6e15ec29f4ebe7c9d
Cr-Commit-Position: refs/heads/master@{#387701}
==========

[commit-bot: I haz the power, 2016-04-15 21:29:39]

Patchset 3 (id:??) landed as
https://crrev.com/59dcea480d282e78805de1c6e15ec29f4ebe7c9d
Cr-Commit-Position: refs/heads/master@{#387701}