Chromecast: use PlayerTrackerImpl in BrowserCdmCast.

Chromecast: use PlayerTrackerImpl in BrowserCdmCast.

This is a simpler implementation of ::media::BrowserCdm's requirement
to implement ::media::PlayerTracker. MediaDrmBridge (Android) does the
same thing.

R=lcwu@chromium.org
BUG=None

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

[lcwu1, 2015-01-08 18:25:17]

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
File chromecast/media/cdm/browser_cdm_cast.cc (right):

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
chromecast/media/cdm/browser_cdm_cast.cc:10: #include
"base/message_loop/message_loop_proxy.h"
No longer needed.

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
File chromecast/media/cdm/browser_cdm_cast.h (right):

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
chromecast/media/cdm/browser_cdm_cast.h:33: virtual ~BrowserCdmCast() override;
Remove virtual.

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
chromecast/media/cdm/browser_cdm_cast.h:47: // Notifies all listeners a new key
was added on the next message loop cycle.
I think this comment is out of date.

[gunsch, 2015-01-08 18:34:46]

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
File chromecast/media/cdm/browser_cdm_cast.cc (right):

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
chromecast/media/cdm/browser_cdm_cast.cc:10: #include
"base/message_loop/message_loop_proxy.h"
On 2015/01/08 18:25:17, lcwu1 wrote:
> No longer needed.

Done.

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
File chromecast/media/cdm/browser_cdm_cast.h (right):

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
chromecast/media/cdm/browser_cdm_cast.h:33: virtual ~BrowserCdmCast() override;
Done via rebase (old CL).

https://codereview.chromium.org/835963005/diff/1/chromecast/media/cdm/browser...
chromecast/media/cdm/browser_cdm_cast.h:47: // Notifies all listeners a new key
was added on the next message loop cycle.
On 2015/01/08 18:25:17, lcwu1 wrote:
> I think this comment is out of date.

Done.

[lcwu1, 2015-01-08 18:42:59]

lgtm % removing unnecessary headers.

https://codereview.chromium.org/835963005/diff/20001/chromecast/media/cdm/bro...
File chromecast/media/cdm/browser_cdm_cast.cc (right):

https://codereview.chromium.org/835963005/diff/20001/chromecast/media/cdm/bro...
chromecast/media/cdm/browser_cdm_cast.cc:10: #include "base/stl_util.h"
I think both location.h and stil_util.h are not needed either.

[gunsch, 2015-01-08 18:58:32]

https://codereview.chromium.org/835963005/diff/20001/chromecast/media/cdm/bro...
File chromecast/media/cdm/browser_cdm_cast.cc (right):

https://codereview.chromium.org/835963005/diff/20001/chromecast/media/cdm/bro...
chromecast/media/cdm/browser_cdm_cast.cc:10: #include "base/stl_util.h"
Done: Looks like none of these are needed with now, actually

[gunsch, 2015-01-08 18:58:38]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2015-01-08 19:00:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/835963005/40001

[commit-bot: I haz the power, 2015-01-08 19:46:21]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-01-08 19:47:04]

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