Delete SurfaceTexturePeer

Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

This change frees 1.7 KiB.

BUG=570711, 339191
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

Review-Url: https://codereview.chromium.org/2420083004
Cr-Commit-Position: refs/heads/master@{#449432}
Committed: https://chromium.googlesource.com/chromium/src/+/882a7c5b519cb21c62a0fffcf36a5cc0a078b903

[tguilbert, 2016-10-15 01:35:55]

Description was changed from

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

BUG=570711
==========

to

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

BUG=570711
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
==========

[tguilbert, 2016-10-15 01:36:13]

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

[commit-bot: I haz the power, 2016-10-15 01:36:33]

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-10-15 03:34:01]

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

[commit-bot: I haz the power, 2016-10-15 03:34:02]

Dry run: This issue passed the CQ dry run.

[tguilbert, 2017-02-07 03:31:47]

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

[commit-bot: I haz the power, 2017-02-07 03:32:22]

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, 2017-02-07 03:57:59]

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

[commit-bot: I haz the power, 2017-02-07 03:58:00]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[tguilbert, 2017-02-07 20:21:18]

Patchset #3 (id:40001) has been deleted

[tguilbert, 2017-02-07 20:25:50]

Patchset #3 (id:60001) has been deleted

[tguilbert, 2017-02-07 20:28:47]

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

[commit-bot: I haz the power, 2017-02-07 20:29:30]

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

[tguilbert, 2017-02-07 20:29:34]

Description was changed from

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

BUG=570711
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

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

BUG=570711, 339191
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
==========

[tguilbert, 2017-02-07 20:53:06]

Description was changed from

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

BUG=570711, 339191
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

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

This change frees 1.7 KiB.

BUG=570711, 339191
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
==========

[tguilbert, 2017-02-07 21:36:11]

tguilbert@chromium.org changed reviewers:
+ kinuko@chromium.org, palmer@chromium.org, watk@chromium.org

[tguilbert, 2017-02-07 21:36:12]

Hello, PTAL :)

watk@: general review.
kinuko@: OWNERS for content/*
palmer@: security/OWNERS for gpu/ipc/common/* and *.aidl
jbauman@: OWNERS for gpu/ipc/service/*

Thank you!
Thomas

[watk, 2017-02-07 22:09:25]

Nice, lgtm

[commit-bot: I haz the power, 2017-02-07 22:21:30]

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

[commit-bot: I haz the power, 2017-02-07 22:21:31]

Dry run: This issue passed the CQ dry run.

[kinuko, 2017-02-07 23:29:11]

lgtm!

[tguilbert, 2017-02-09 20:20:43]

On 2017/02/07 23:29:11, kinuko wrote:
> lgtm!

friendly ping :)

[tguilbert, 2017-02-09 20:20:43]

On 2017/02/07 23:29:11, kinuko wrote:
> lgtm!

friendly ping :)

[palmer, 2017-02-09 20:30:00]

RS LGTM

[tguilbert, 2017-02-09 20:33:12]

On 2017/02/09 20:30:00, palmer wrote:
> RS LGTM

Sorry for the double ping. Thank you for the review!

[tguilbert, 2017-02-09 20:33:18]

The CQ bit was checked by tguilbert@chromium.org

[commit-bot: I haz the power, 2017-02-09 20:34:03]

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, 2017-02-09 20:41:51]

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

[commit-bot: I haz the power, 2017-02-09 20:41:52]

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...)

[tguilbert, 2017-02-09 21:16:18]

tguilbert@chromium.org changed reviewers:
+ jbauman@chromium.org

[tguilbert, 2017-02-09 21:16:19]

I forgot some aliases in my initial message.

jbauman@, can you OWNERS review gpu/ipc/service?

Thanks!
Thomas

[jbauman, 2017-02-09 21:33:06]

lgtm

[tguilbert, 2017-02-09 21:59:17]

The CQ bit was checked by tguilbert@chromium.org

[commit-bot: I haz the power, 2017-02-09 21:59: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, 2017-02-09 22:05:52]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1486677557398200,
"parent_rev": "ec84e4ea604deb4d6cca4b72ce6f97babf99878b", "commit_rev":
"882a7c5b519cb21c62a0fffcf36a5cc0a078b903"}

[commit-bot: I haz the power, 2017-02-09 22:06:29]

Description was changed from

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

This change frees 1.7 KiB.

BUG=570711, 339191
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

==========
Delete SurfaceTexturePeer

With WMPA deleted, there is no more use for SurfaceTexturePeer.

Use ScopedSurfaceRequestManager instead.

This change frees 1.7 KiB.

BUG=570711, 339191
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

Review-Url: https://codereview.chromium.org/2420083004
Cr-Commit-Position: refs/heads/master@{#449432}
Committed:
https://chromium.googlesource.com/chromium/src/+/882a7c5b519cb21c62a0fffcf36a...
==========

[commit-bot: I haz the power, 2017-02-09 22:06:30]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/882a7c5b519cb21c62a0fffcf36a...