Remove SurfaceFactory::Reset

Remove SurfaceFactory::Reset

It's not used anymore. We instead recreate the 
CompositorFrameSinkSupport object on gpu restart. This is because
the CompositorFrameSinkSupport will eventually live in the gpu process 
and will be destroyed on gpu restart, therefore it can't have a Reset
method like SurfaceFactory.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2757563003
Cr-Commit-Position: refs/heads/master@{#457453}
Committed: https://chromium.googlesource.com/chromium/src/+/ed13215635f34ea3026cbb0da8c91dad903815dc

[Saman Sami, 2017-03-16 15:02:03]

Description was changed from

==========
Remove SurfaceFactory::Reset

It's not used anymore.
==========

to

==========
Remove SurfaceFactory::Reset

It's not used anymore.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Saman Sami, 2017-03-16 15:02:04]

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

[Saman Sami, 2017-03-16 15:02:24]

Description was changed from

==========
Remove SurfaceFactory::Reset

It's not used anymore.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove SurfaceFactory::Reset

It's not used anymore.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Saman Sami, 2017-03-16 15:02:25]

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

[commit-bot: I haz the power, 2017-03-16 15:02:43]

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

[Saman Sami, 2017-03-16 15:02:51]

PTAL

[Saman Sami, 2017-03-16 15:02:59]

Description was changed from

==========
Remove SurfaceFactory::Reset

It's not used anymore.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove SurfaceFactory::Reset

It's not used anymore.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Fady Samuel, 2017-03-16 15:04:40]

LGTM, but please ask piman@ to review this since he wrote it.

[Saman Sami, 2017-03-16 15:05:09]

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

[Saman Sami, 2017-03-16 15:05:30]

Thanks!

piman@ PTAL

[Saman Sami, 2017-03-16 15:07:18]

Description was changed from

==========
Remove SurfaceFactory::Reset

It's not used anymore.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove SurfaceFactory::Reset

It's not used anymore. We instead recreate the 
CompositorFrameSinkSupport object on gpu restart. This is because
the CompositorFrameSinkSupport will eventually live in the gpu process 
and will be destroyed on gpu restart, therefore it can't have a Reset
method like SurfaceFactory.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[piman, 2017-03-16 15:54:18]

On 2017/03/16 15:05:30, Saman Sami wrote:
> Thanks!
> 
> piman@ PTAL

LGTM. It's not used anymore, I assume it was replaced by something appropriate

[commit-bot: I haz the power, 2017-03-16 16:04:57]

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

[commit-bot: I haz the power, 2017-03-16 16:04:57]

Dry run: This issue passed the CQ dry run.

[Saman Sami, 2017-03-16 16:13:56]

The CQ bit was checked by samans@chromium.org

[commit-bot: I haz the power, 2017-03-16 16:14:29]

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-03-16 16:19:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489680836489050, "parent_rev":
"d36229e8166288e84af7485819ceeb1a347465a9", "commit_rev":
"ed13215635f34ea3026cbb0da8c91dad903815dc"}

[commit-bot: I haz the power, 2017-03-16 16:19:57]

Description was changed from

==========
Remove SurfaceFactory::Reset

It's not used anymore. We instead recreate the 
CompositorFrameSinkSupport object on gpu restart. This is because
the CompositorFrameSinkSupport will eventually live in the gpu process 
and will be destroyed on gpu restart, therefore it can't have a Reset
method like SurfaceFactory.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove SurfaceFactory::Reset

It's not used anymore. We instead recreate the 
CompositorFrameSinkSupport object on gpu restart. This is because
the CompositorFrameSinkSupport will eventually live in the gpu process 
and will be destroyed on gpu restart, therefore it can't have a Reset
method like SurfaceFactory.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2757563003
Cr-Commit-Position: refs/heads/master@{#457453}
Committed:
https://chromium.googlesource.com/chromium/src/+/ed13215635f34ea3026cbb0da8c9...
==========

[commit-bot: I haz the power, 2017-03-16 16:19:58]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ed13215635f34ea3026cbb0da8c9...