Drop [LegacyInterfaceTypeChecking] where trivial in core/

Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.

BUG=561338
Committed: https://crrev.com/47ce5feb1d13ce889da15f76f063a7978a0b5c37
Cr-Commit-Position: refs/heads/master@{#362117}

[philipj_slow, 2015-11-30 09:36:23]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

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

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.
BUG=561338
==========

[philipj_slow, 2015-11-30 09:36:31]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.
BUG=561338
==========

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.

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

[philipj_slow, 2015-11-30 09:37:19]

philipj@opera.com changed reviewers:
+ haraken@chromium.org

[philipj_slow, 2015-11-30 09:37:20]

PTAL

[haraken, 2015-11-30 09:42:27]

LGTM

[philipj_slow, 2015-11-30 09:43:29]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-11-30 09:43:51]

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

[commit-bot: I haz the power, 2015-11-30 10:57:05]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.

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

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.

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

[commit-bot: I haz the power, 2015-11-30 10:57:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-30 10:57:49]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.

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

to

==========
Drop [LegacyInterfaceTypeChecking] where trivial in core/

These are cases where passing null to the implementation would already
throw an exception, and so letting the bindings code do it is cleaner.
In a few cases this means that the implementation no longer throws any
exceptions.

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 are from bindings.

BUG=561338

Committed: https://crrev.com/47ce5feb1d13ce889da15f76f063a7978a0b5c37
Cr-Commit-Position: refs/heads/master@{#362117}
==========

[commit-bot: I haz the power, 2015-11-30 10:57:49]

Patchset 3 (id:??) landed as
https://crrev.com/47ce5feb1d13ce889da15f76f063a7978a0b5c37
Cr-Commit-Position: refs/heads/master@{#362117}