Tweak the public/protected/private sections of MediaControls

Tweak the public/protected/private sections of MediaControls

After the merge this looked rather strange, so keep as public that
which is used by HTMLMediaElement or MediaControlElements, make
protected that which is used/overridden by MediaControlsAndroid, and
make everything else private.

BUG=341813
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=166911

[philipj_slow, 2014-02-07 13:30:17]

Patch 4, depends on https://codereview.chromium.org/141323005/

https://codereview.chromium.org/130693008/ is all of the patches together in a
trybot run to get some confidence that this will work.

[acolwell GONE FROM CHROMIUM, 2014-02-07 22:08:40]

lgtm

https://codereview.chromium.org/143703007/diff/1/Source/core/html/shadow/Medi...
File Source/core/html/shadow/MediaControls.h (right):

https://codereview.chromium.org/143703007/diff/1/Source/core/html/shadow/Medi...
Source/core/html/shadow/MediaControls.h:102: virtual bool
willRespondToMouseMoveEvents() OVERRIDE { return true; }
nit: Please add a comment indicating that these are Node methods.

[philipj_slow, 2014-02-08 04:00:24]

https://codereview.chromium.org/143703007/diff/1/Source/core/html/shadow/Medi...
File Source/core/html/shadow/MediaControls.h (right):

https://codereview.chromium.org/143703007/diff/1/Source/core/html/shadow/Medi...
Source/core/html/shadow/MediaControls.h:102: virtual bool
willRespondToMouseMoveEvents() OVERRIDE { return true; }
On 2014/02/07 22:08:40, acolwell wrote:
> nit: Please add a comment indicating that these are Node methods.

Done.

[philipj_slow, 2014-02-11 09:45:04]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-02-11 09:45:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/143703007/80001

[commit-bot: I haz the power, 2014-02-11 11:00:58]

Change committed as 166911