Deprecate subscriptions to MediaStream ended event

Deprecate subscriptions to MediaStream ended event including those made with addEventListener.

Intent to deprecate and remove: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/EHy8zm0eVy0

BUG=608795
Committed: https://crrev.com/fcbbc1cb8bb9ce60e7bb29a7f1aff02f06be412d
Cr-Commit-Position: refs/heads/master@{#392232}

[Guido Urdaneta, 2016-05-05 10:39:51]

Description was changed from

==========
Count uses MediaStream ended event subscriptions made with addEventListener.

BUG=608795
==========

to

==========
Count uses MediaStream ended event subscriptions made with addEventListener.

BUG=608795
==========

[Guido Urdaneta, 2016-05-05 10:39:51]

guidou@chromium.org changed reviewers:
+ rbyers@chromium.org

[Guido Urdaneta, 2016-05-05 10:45:00]

guidou@chromium.org changed reviewers:
+ tommi@chromium.org

[Guido Urdaneta, 2016-05-05 10:45:00]

Hi, PTAL

[Guido Urdaneta, 2016-05-05 10:45:30]

guidou@chromium.org changed reviewers:
- tommi@chromium.org

[Guido Urdaneta, 2016-05-05 10:46:14]

Description was changed from

==========
Count uses MediaStream ended event subscriptions made with addEventListener.

BUG=608795
==========

to

==========
Count uses of MediaStream ended event subscriptions made with addEventListener.

BUG=608795
==========

[Rick Byers, 2016-05-05 14:08:58]

LGTM with nit

You're going to also turn this into a deprecation (instead of just a counter)
sometime in M52, right?

https://codereview.chromium.org/1954693002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/mediastream/MediaStream.cpp (right):

https://codereview.chromium.org/1954693002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/mediastream/MediaStream.cpp:310: return
EventTarget::addEventListenerInternal(eventType, listener, options);
nit: This should probably be
"EventTargetWithInlineData::addEventListenerInternal" since this class inherits
from EventTargetWithInlineData (although there's obviously no difference right
now, but just incase we ever overrode addEventListenerInternal in
EventTargetWithInlineData.

[Guido Urdaneta, 2016-05-06 12:47:57]

Description was changed from

==========
Count uses of MediaStream ended event subscriptions made with addEventListener.

BUG=608795
==========

to

==========
Deprecate subscriptions to MediaStream ended event including those made with
addEventListener.

Intent to deprecate and remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/EHy8zm0eVy0

BUG=608795
==========

[Guido Urdaneta, 2016-05-06 12:49:49]

Hi, I turned this CL into the deprecation CL. PTAL.

[Rick Byers, 2016-05-06 17:54:40]

LGTM

[Guido Urdaneta, 2016-05-06 18:08:58]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2016-05-06 18:09:25]

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

[commit-bot: I haz the power, 2016-05-06 18:20:49]

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

[commit-bot: I haz the power, 2016-05-06 18:20:50]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)

[Guido Urdaneta, 2016-05-06 18:59:47]

The CQ bit was checked by guidou@chromium.org

[Guido Urdaneta, 2016-05-06 18:59:47]

The patchset sent to the CQ was uploaded after l-g-t-m from rbyers@chromium.org
Link to the patchset: https://codereview.chromium.org/1954693002/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-06 19:00:07]

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

[commit-bot: I haz the power, 2016-05-06 22:00:37]

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

[commit-bot: I haz the power, 2016-05-06 22:00:38]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Guido Urdaneta, 2016-05-06 22:19:16]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2016-05-06 22:19:46]

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

[commit-bot: I haz the power, 2016-05-07 01:49:10]

Description was changed from

==========
Deprecate subscriptions to MediaStream ended event including those made with
addEventListener.

Intent to deprecate and remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/EHy8zm0eVy0

BUG=608795
==========

to

==========
Deprecate subscriptions to MediaStream ended event including those made with
addEventListener.

Intent to deprecate and remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/EHy8zm0eVy0

BUG=608795
==========

[commit-bot: I haz the power, 2016-05-07 01:49:11]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-05-07 01:50:24]

Description was changed from

==========
Deprecate subscriptions to MediaStream ended event including those made with
addEventListener.

Intent to deprecate and remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/EHy8zm0eVy0

BUG=608795
==========

to

==========
Deprecate subscriptions to MediaStream ended event including those made with
addEventListener.

Intent to deprecate and remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/EHy8zm0eVy0

BUG=608795

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

[commit-bot: I haz the power, 2016-05-07 01:50:25]

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