Remove IOSurfaceLayer

Remove IOSurfaceLayer

This was only used by NPAPI.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/3a5e993ac76ffb3bfbb359f3dbd9516574e982e7
Cr-Commit-Position: refs/heads/master@{#386450}

[piman, 2016-04-08 02:40:59]

Description was changed from

==========
Remove IOSurfaceLayer

This was only used by NPAPI.

BUG=
==========

to

==========
Remove IOSurfaceLayer

This was only used by NPAPI.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[piman, 2016-04-08 02:41:09]

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

[piman, 2016-04-08 03:45:38]

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

[commit-bot: I haz the power, 2016-04-08 03:45:53]

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

[commit-bot: I haz the power, 2016-04-08 04:38:28]

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

[commit-bot: I haz the power, 2016-04-08 04:38:29]

Dry run: This issue passed the CQ dry run.

[danakj, 2016-04-08 20:45:12]

Oh wow cool LGTM

https://codereview.chromium.org/1869983002/diff/1/cc/trees/layer_tree_host_un...
File cc/trees/layer_tree_host_unittest.cc (left):

https://codereview.chromium.org/1869983002/diff/1/cc/trees/layer_tree_host_un...
cc/trees/layer_tree_host_unittest.cc:2284: test_capabilities_.gpu.iosurface =
true;
Do either of these caps become unused/deletable?

[danakj, 2016-04-08 20:45:32]

https://code.google.com/p/chromium/codesearch#chromium/src/cc/quads/io_surfac...
too?

[piman, 2016-04-11 18:17:33]

https://codereview.chromium.org/1869983002/diff/1/cc/trees/layer_tree_host_un...
File cc/trees/layer_tree_host_unittest.cc (left):

https://codereview.chromium.org/1869983002/diff/1/cc/trees/layer_tree_host_un...
cc/trees/layer_tree_host_unittest.cc:2284: test_capabilities_.gpu.iosurface =
true;
On 2016/04/08 20:45:12, danakj wrote:
> Do either of these caps become unused/deletable?

Nope. IOSurfaceDrawQuad is still used by VideoLayerImpl (and rely on these
caps). There was a CL to remove it, but it got reverted - see
https://codereview.chromium.org/1162443005

Maybe we should try to push it further, since GpuMemoryBuffer is really the long
term thing, but that'll have to be another CL.

[piman, 2016-04-11 18:17:40]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2016-04-11 18:18:11]

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

[commit-bot: I haz the power, 2016-04-11 18:21:46]

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

[commit-bot: I haz the power, 2016-04-11 18:21:47]

Try jobs failed on following builders:
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  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_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[piman, 2016-04-11 18:35:57]

The CQ bit was checked by piman@chromium.org

[piman, 2016-04-11 18:35:57]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/1869983002/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2016-04-11 18:36:46]

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

[commit-bot: I haz the power, 2016-04-11 20:05:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-11 20:06:47]

Description was changed from

==========
Remove IOSurfaceLayer

This was only used by NPAPI.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Remove IOSurfaceLayer

This was only used by NPAPI.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/3a5e993ac76ffb3bfbb359f3dbd9516574e982e7
Cr-Commit-Position: refs/heads/master@{#386450}
==========

[commit-bot: I haz the power, 2016-04-11 20:06:48]

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