Remove SurfaceReferenceManager interface.

Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

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

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

[kylechar, 2017-01-24 19:02:17]

Description was changed from

==========
Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

BUG=659227
==========

to

==========
Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

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

[kylechar, 2017-01-25 19:41:27]

Description was changed from

==========
Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

BUG=659227
==========

to

==========
Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

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

[kylechar, 2017-01-25 19:44:32]

Patchset #2 (id:20001) has been deleted

[kylechar, 2017-01-25 19:47:12]

kylechar@chromium.org changed reviewers:
+ fsamuel@chromium.org, jbauman@chromium.org

[kylechar, 2017-01-25 19:47:12]

jbauman: cc/surfaces/*
fsamuel: services/ui/surfaces/*

[Fady Samuel, 2017-01-25 19:50:02]

Thanks for the cleanup! LGTM

[xing.xu, 2017-01-26 01:43:46]

LGTM. BTW, this may conflicts with https://codereview.chromium.org/2654693003/

[jbauman, 2017-01-26 02:08:08]

lgtm

[kylechar, 2017-01-27 15:35:42]

Patchset #3 (id:60001) has been deleted

[kylechar, 2017-01-27 15:37:07]

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

[commit-bot: I haz the power, 2017-01-27 15:37:23]

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-01-27 16:34:28]

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

[commit-bot: I haz the power, 2017-01-27 16:34:29]

Dry run: This issue passed the CQ dry run.

[kylechar, 2017-01-27 16:35:11]

Thanks!

[kylechar, 2017-01-27 16:35:19]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2017-01-27 16:35:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
fsamuel@chromium.org, jbauman@chromium.org, xing.xu@intel.com
Link to the patchset: https://codereview.chromium.org/2654653004/#ps80001
(title: "Rebase + fix tests.")

[commit-bot: I haz the power, 2017-01-27 16:35: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, 2017-01-27 16:45:31]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1485534919280350,
"parent_rev": "f329d552a1d32251a9c20ba06183fe5aa203040d", "commit_rev":
"a5af59cbbaccb5f6705e2e3b6c62984c21a398e1"}

[commit-bot: I haz the power, 2017-01-27 16:46:08]

Description was changed from

==========
Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

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

to

==========
Remove SurfaceReferenceManager interface.

The interface is no longer necessary for tests. Delete the interface and
move method comments to SurfaceManager. Cleanup the last references to
the interface in DisplayCompositor.

Also removes the GetSurfaceReferenceCount() / GetReferenceSurfaceCount()
methods that are unused. We don't care about the number of references,
we care about reachable from the top-level root.

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

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

[commit-bot: I haz the power, 2017-01-27 16:46:09]

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