Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

The extra type checking in the bindings of the MediaStream constructor
and RTCDataChannel's send() methods is actually not observable, because
there is type testing to determine which override to use that makes sure
those code paths aren't reached if the types aren't correct.

For each method affected, it was verified using cs.chromium.org that
there are no internal calls where nullptr could be passed, the only
calls were from bindings an unit tests.

BUG=561338
Committed: https://crrev.com/027253cdbbe06eb7c2352b5146c4f5a60515940c
Cr-Commit-Position: refs/heads/master@{#362024}

[philipj_slow, 2015-11-27 14:13:13]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

BUG=561338
==========

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

The extra type checking in the bindings of the MediaStream constructor
and RTCDataChannel's send() methods is actually not observable, because
there is type testing to determine which override to use that makes sure
those code paths aren't reached if the types aren't correct.

For each method affected, it was verified using cs.chromium.org that
there are no internal calls where nullptr could be passed, the only
calls were from bindings an unit tests.

BUG=561338
==========

[philipj_slow, 2015-11-27 14:31:36]

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

[philipj_slow, 2015-11-27 14:32:44]

philipj@opera.com changed reviewers:
+ guidou@chromium.org, hta@chromium.org, peter@chromium.org, tommi@chromium.org

[philipj_slow, 2015-11-27 14:32:45]

PTAL. More reviewers than usual because I suspect that hta@ and guidou@ will be
most interested in this, but they are not owners (yet?)

[commit-bot: I haz the power, 2015-11-27 14:33:16]

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

[Guido Urdaneta, 2015-11-27 14:39:32]

lgtm

[Peter Beverloo, 2015-11-27 14:42:49]

On 2015/11/27 14:39:32, Guido Urdaneta wrote:
> lgtm

Stampedy stamp lgtm - thanks for the quick look, and thanks Phillip for the
patch :)

[commit-bot: I haz the power, 2015-11-27 15:46:49]

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

[commit-bot: I haz the power, 2015-11-27 15:46:50]

Dry run: This issue passed the CQ dry run.

[tommi (sloooow) - chröme, 2015-11-27 17:17:45]

rs lgtm

[philipj_slow, 2015-11-27 17:57:28]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-11-27 17:57:49]

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

[commit-bot: I haz the power, 2015-11-27 18:03:49]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

The extra type checking in the bindings of the MediaStream constructor
and RTCDataChannel's send() methods is actually not observable, because
there is type testing to determine which override to use that makes sure
those code paths aren't reached if the types aren't correct.

For each method affected, it was verified using cs.chromium.org that
there are no internal calls where nullptr could be passed, the only
calls were from bindings an unit tests.

BUG=561338
==========

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

The extra type checking in the bindings of the MediaStream constructor
and RTCDataChannel's send() methods is actually not observable, because
there is type testing to determine which override to use that makes sure
those code paths aren't reached if the types aren't correct.

For each method affected, it was verified using cs.chromium.org that
there are no internal calls where nullptr could be passed, the only
calls were from bindings an unit tests.

BUG=561338
==========

[commit-bot: I haz the power, 2015-11-27 18:03:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-27 18:04:50]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

The extra type checking in the bindings of the MediaStream constructor
and RTCDataChannel's send() methods is actually not observable, because
there is type testing to determine which override to use that makes sure
those code paths aren't reached if the types aren't correct.

For each method affected, it was verified using cs.chromium.org that
there are no internal calls where nullptr could be passed, the only
calls were from bindings an unit tests.

BUG=561338
==========

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in WebRTC

The extra type checking in the bindings of the MediaStream constructor
and RTCDataChannel's send() methods is actually not observable, because
there is type testing to determine which override to use that makes sure
those code paths aren't reached if the types aren't correct.

For each method affected, it was verified using cs.chromium.org that
there are no internal calls where nullptr could be passed, the only
calls were from bindings an unit tests.

BUG=561338

Committed: https://crrev.com/027253cdbbe06eb7c2352b5146c4f5a60515940c
Cr-Commit-Position: refs/heads/master@{#362024}
==========

[commit-bot: I haz the power, 2015-11-27 18:04:51]

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