[media] Removed unused method declaration from WMPI

[media] Removed unused method declaration from WMPI

Seems to be a leftover from some old refactoring.

BUG=None
TEST=built chrome and media_unittests

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

[whywhat, 2016-11-23 03:32:42]

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

[commit-bot: I haz the power, 2016-11-23 03:32:58]

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

[whywhat, 2016-11-23 03:34:13]

avayvod@chromium.org changed reviewers:
+ sandersd@chromium.org

[whywhat, 2016-11-23 03:34:14]

+Dan

PTaL

Is it used anywhere where Code Search wouldn't find it, like Chromecast or
anything else?
More importantly, where does WMPI learn now that there's a new frame painted?

[commit-bot: I haz the power, 2016-11-23 04:50:48]

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

[commit-bot: I haz the power, 2016-11-23 04:50:49]

Dry run: This issue passed the CQ dry run.

[sandersd (OOO until July 31), 2016-11-23 19:02:05]

On 2016/11/23 03:34:14, whywhat wrote:
> +Dan
> 
> PTaL
> 
> Is it used anywhere where Code Search wouldn't find it, like Chromecast or
> anything else?
> More importantly, where does WMPI learn now that there's a new frame painted?

It looks like FrameReady() has been gone for over two years; the mechanism to
actually paint frames has been replaced twice since then.

The short answer to your question is that WMPI doesn't. VideoRendererAlgorithm
is responsible for all of those things, but it actually works in a poll model
where the compositor asks for frames on vsync. The only compositing-related
notification that VideoRendererAlgorithm gets is for dropped frames.


lgtm.

[whywhat, 2016-11-23 20:38:52]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2016-11-23 20:39:37]

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-11-23 21:44:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479933532964050, "parent_rev":
"d77d87db0090a3795a8113494d8103912ace7a82", "commit_rev":
"5a8583b949fd401e97655f7535531d7379919c4d"}

[commit-bot: I haz the power, 2016-11-23 21:44:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-23 21:48:26]

Description was changed from

==========
[media] Removed unused method declaration from WMPI

Seems to be a leftover from some old refactoring.

BUG=None
TEST=built chrome and media_unittests
==========

to

==========
[media] Removed unused method declaration from WMPI

Seems to be a leftover from some old refactoring.

BUG=None
TEST=built chrome and media_unittests

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

[commit-bot: I haz the power, 2016-11-23 21:48:27]

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