Remove MediaController (already diabled by REF)

Remove MediaController (already diabled by REF)

Intent to Remove:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/66zec0JPy-k/-89CRX4VBQAJ

It was put behind a disabled RuntimeEnabledFeature in 2014:
https://codereview.chromium.org/250553006

BUG=none

Committed: https://crrev.com/2f5ffbda7e294845fe59be6fd4aaa74dec5d3cc6
Cr-Commit-Position: refs/heads/master@{#355260}

[philipj_slow, 2015-10-02 13:45:58]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-10-02 13:47:06]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1373423003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1373423003/1

[commit-bot: I haz the power, 2015-10-02 14:50:47]

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

[commit-bot: I haz the power, 2015-10-02 14:50:47]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[philipj_slow, 2015-10-07 10:05:16]

philipj@opera.com changed reviewers:
+ fs@opera.com

[philipj_slow, 2015-10-07 10:05:17]

PTAL

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLMediaElement.cpp (right):

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLMediaElement.cpp:2710: // If the media
element has a loop attribute specified
I'm changing the comments in the way I imagine the spec would change. Good or
bad? Maybe leave the comments along until the spec changes?

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLMediaElement.cpp:2716: // If the media
element
Should rewrap this if not reverted to previous wording.

[fs, 2015-10-07 10:17:14]

lgtm

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLMediaElement.cpp (right):

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLMediaElement.cpp:2710: // If the media
element has a loop attribute specified
On 2015/10/07 at 10:05:17, philipj wrote:
> I'm changing the comments in the way I imagine the spec would change. Good or
bad? Maybe leave the comments along until the spec changes?

I guess you could just leave an indication that there's a difference or
something. Any way is fine by me.

[philipj_slow, 2015-10-07 13:21:13]

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLMediaElement.cpp (right):

https://codereview.chromium.org/1373423003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLMediaElement.cpp:2710: // If the media
element has a loop attribute specified
On 2015/10/07 10:17:14, fs wrote:
> On 2015/10/07 at 10:05:17, philipj wrote:
> > I'm changing the comments in the way I imagine the spec would change. Good
or
> bad? Maybe leave the comments along until the spec changes?
> 
> I guess you could just leave an indication that there's a difference or
> something. Any way is fine by me.

I had a look at the spec and noticed other differences, namely that this should
all be in a queued task. Trying to annotate the microscopic differences here
likely won't help anyone get this in sync with the spec, so I just left the
changes I think I'd make to the spec text instead and rewrapped it.

[philipj_slow, 2015-10-07 13:21:18]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-10-07 13:21:18]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1373423003/#ps20001
(title: "rewrap comment")

[commit-bot: I haz the power, 2015-10-07 13:23:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1373423003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1373423003/20001

[commit-bot: I haz the power, 2015-10-07 14:17:13]

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

[commit-bot: I haz the power, 2015-10-07 14:17:14]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[philipj_slow, 2015-10-21 07:24:56]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-10-21 07:24:57]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1373423003/#ps60001
(title: "update web-platform-tests expectations")

[commit-bot: I haz the power, 2015-10-21 07:25:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1373423003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1373423003/60001

[commit-bot: I haz the power, 2015-10-21 08:27:49]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-21 08:28:35]

Patchset 4 (id:??) landed as
https://crrev.com/2f5ffbda7e294845fe59be6fd4aaa74dec5d3cc6
Cr-Commit-Position: refs/heads/master@{#355260}