[Refactor] Introduce MediaDelegate

Introduce MediaDelegate

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232330

[oshima, 2013-10-31 06:58:06]

Steven, would you mind reviewing this, as the original author (sschmitz) is
gone?

[stevenjb, 2013-10-31 20:40:03]

lgtm w/ protected or DISALLOW

https://codereview.chromium.org/48523010/diff/40001/ash/media_delegate.h
File ash/media_delegate.h (right):

https://codereview.chromium.org/48523010/diff/40001/ash/media_delegate.h#newc...
ash/media_delegate.h:14: virtual ~MediaDelegate() {}
Protect destructor or use DISALLOW...
nit: constructor unnecessary

[oshima, 2013-11-01 00:30:25]

let me know if you still think i should add DISALLOW..

https://codereview.chromium.org/48523010/diff/40001/ash/media_delegate.h
File ash/media_delegate.h (right):

https://codereview.chromium.org/48523010/diff/40001/ash/media_delegate.h#newc...
ash/media_delegate.h:14: virtual ~MediaDelegate() {}
On 2013/10/31 20:40:03, stevenjb wrote:
> Protect destructor or use DISALLOW...

owned by shell, so can't be protected, and i believe
pur interface doesn't need DISALLOW_ (confirmed 
window_observer/delegate, shell_delegate and others don't have it)

> nit: constructor unnecessary

Done.

[commit-bot: I haz the power, 2013-11-01 00:37:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/48523010/120001

[commit-bot: I haz the power, 2013-11-01 11:43:15]

Change committed as 232330