ui: Remove SurfaceTexture GpuMemoryBuffer type.

ui: Remove SurfaceTexture GpuMemoryBuffer type.

This is not currently used and there are no plans to enable it.
Remove it for now to keep the code simple. We can always add it
back later if it turns out useful.

BUG=
TEST=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[reveman, 2016-09-18 19:46:54]

Description was changed from

==========
ui: Remove SurfaceTexture GpuMemoryBuffer type.

This is not currently used and there are no plans to enable it.
Remove it for now to keep the code simple. We can always add it
back later if it turns out useful.

BUG=
TEST=
==========

to

==========
ui: Remove SurfaceTexture GpuMemoryBuffer type.

This is not currently used and there are no plans to enable it.
Remove it for now to keep the code simple. We can always add it
back later if it turns out useful.

BUG=
TEST=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[reveman, 2016-09-18 19:48:40]

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

[commit-bot: I haz the power, 2016-09-18 19:48:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-18 19:58:33]

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

[commit-bot: I haz the power, 2016-09-18 19:58:34]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[reveman, 2016-09-18 20:14:22]

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

[commit-bot: I haz the power, 2016-09-18 20:14:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-18 20:22:41]

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

[commit-bot: I haz the power, 2016-09-18 20:22:43]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[reveman, 2016-09-18 20:33:00]

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

[commit-bot: I haz the power, 2016-09-18 20:33:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-18 21:44:53]

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

[commit-bot: I haz the power, 2016-09-18 21:44:54]

Dry run: This issue passed the CQ dry run.

[reveman, 2016-09-18 22:23:28]

reveman@chromium.org changed reviewers:
+ aelias@chromium.org, piman@chromium.org

[aelias_OOO_until_Jul13, 2016-09-19 14:47:18]

lgtm

[DaleCurtis, 2016-09-19 18:43:00]

dalecurtis@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2016-09-19 18:43:00]

What were the reasons this went unused? We have libvpx decoding in Chrome now,
so it might be worth trying to see if things are improved when we fallback to
internal software decoders.

[aelias_OOO_until_Jul13, 2016-09-19 19:08:28]

It hasn't provided any performance improvement over a more normal kind of
buffer, nor can we come up with a use case that might.  For video purposes,
you're probably more interested in a SurfaceTexture buffer queue, not a single
buffer like what this provides.

[DaleCurtis, 2016-09-19 19:13:02]

dalecurtis@chromium.org changed reviewers:
- dalecurtis@chromium.org

[DaleCurtis, 2016-09-19 19:13:03]

Ah, yeah each one is a whole SurfaceTexture that's unfortunate.

[piman, 2016-09-19 20:37:44]

LGTM, thanks for the cleanup!

[reveman, 2016-09-19 20:51:39]

On 2016/09/19 at 20:37:44, piman wrote:
> LGTM, thanks for the cleanup!

Thanks for the quick review. More cleanup to follow as a result of this being
removed.

[reveman, 2016-09-19 20:51:46]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-09-19 20:52:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-19 21:02:45]

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

[commit-bot: I haz the power, 2016-09-19 21:02:46]

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

[reveman, 2016-09-19 21:31:49]

reveman@chromium.org changed reviewers:
+ rsesek@chromium.org

[reveman, 2016-09-19 21:31:49]

+rsesek for ui/gfx/mojo (security)

[Robert Sesek, 2016-09-19 22:27:07]

lgtm

[reveman, 2016-09-19 22:42:25]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-09-19 22:42:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-19 22:52:33]

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

[commit-bot: I haz the power, 2016-09-19 22:52:35]

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

[reveman, 2016-09-20 06:57:33]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-09-20 06:57:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-20 07:06:01]

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

[commit-bot: I haz the power, 2016-09-20 07:06:02]

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

[reveman, 2016-09-20 07:58:12]

reveman@chromium.org changed reviewers:
+ danakj@chromium.org

[reveman, 2016-09-20 07:58:12]

+danakj for ui/gfx/mojo/buffer_types_traits.h

fyi, I'll follow up with a patch that adjusts the ui/gfx/OWNER file so GPU
memory buffer ownership includes this file. mojo sub directory is making the
current owner check fail today.

[reveman, 2016-09-21 12:01:20]

reveman@chromium.org changed reviewers:
+ asvitkine@chromium.org

[reveman, 2016-09-21 12:01:21]

+asvitkine for ui/gfx/

[reveman, 2016-09-21 12:01:33]

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

[commit-bot: I haz the power, 2016-09-21 12:01:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-21 15:16:33]

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

[commit-bot: I haz the power, 2016-09-21 15:16:34]

Dry run: Try jobs failed on following builders:
  win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

[Alexei Svitkine (slow), 2016-09-21 15:32:19]

gfx lgtm rubberstamp

[reveman, 2016-09-21 15:48:47]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-09-21 15:49:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-21 17:50:25]

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

[commit-bot: I haz the power, 2016-09-21 17:50:27]

Try jobs failed on following builders:
  win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_TIMED_OUT, no
build URL)

[reveman, 2016-09-21 20:09:08]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-09-21 20:09:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-21 22:53:33]

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

[commit-bot: I haz the power, 2016-09-21 22:53:34]

Try jobs failed on following builders:
  win_optional_gpu_tests_rel on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

[reveman, 2016-09-22 03:11:24]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-09-22 03:11:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-22 04:05:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-22 04:06:13]

Description was changed from

==========
ui: Remove SurfaceTexture GpuMemoryBuffer type.

This is not currently used and there are no plans to enable it.
Remove it for now to keep the code simple. We can always add it
back later if it turns out useful.

BUG=
TEST=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
ui: Remove SurfaceTexture GpuMemoryBuffer type.

This is not currently used and there are no plans to enable it.
Remove it for now to keep the code simple. We can always add it
back later if it turns out useful.

BUG=
TEST=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[commit-bot: I haz the power, 2016-09-22 04:06:15]

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