Drop [LegacyInterfaceTypeChecking] for the Presentation API

Drop [LegacyInterfaceTypeChecking] for the Presentation API

The change to the send() methods are unobservable, because the added
TypeError exceptions in the generated code are unreachable, as type
testing is first used to pick which send() to dispatch to.

The change to defaultRequest is observable, in that something like
`presentation.defaultRequest = {}` will now throw TypeError instead of
setting defaultRequest to null. Since this is a very new API, this is
very unlikely to be a problem.

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

[philipj_slow, 2015-12-04 21:45:48]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] for the Presentation API

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

to

==========
Drop [LegacyInterfaceTypeChecking] for the Presentation API

The change to the send() methods are unobservable, because the added
TypeError exceptions in the generated code are unreachable, as type
testing is first used to pick which send() to dispatch to.

The change to defaultRequest is observable, in that something like
`presentation.defaultRequest = {}` will now throw TypeError instead of
setting defaultRequest to null. Since this is a very new API, this is
very unlikely to be a problem.

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

[philipj_slow, 2015-12-04 21:47:36]

philipj@opera.com changed reviewers:
+ avayvod@chromium.org, imcheng@chromium.org, mfoltz@chromium.org,
mlamouri@chromium.org

[philipj_slow, 2015-12-04 21:47:37]

PTAL. I'm on the fence about adding a test for defaultRequest. Nobody will have
any reason to that for new APIs going forward, and it'll just look like clutter
once everybody has forgotten about [LegacyInterfaceTypeChecking]. If you want a
test, it's trivial and I will add one.

[mlamouri (slow - plz ping), 2015-12-07 10:20:15]

The CQ bit was checked by mlamouri@chromium.org

[mlamouri (slow - plz ping), 2015-12-07 10:20:15]

lgtm

[commit-bot: I haz the power, 2015-12-07 10:20:46]

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

[commit-bot: I haz the power, 2015-12-07 13:04:04]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] for the Presentation API

The change to the send() methods are unobservable, because the added
TypeError exceptions in the generated code are unreachable, as type
testing is first used to pick which send() to dispatch to.

The change to defaultRequest is observable, in that something like
`presentation.defaultRequest = {}` will now throw TypeError instead of
setting defaultRequest to null. Since this is a very new API, this is
very unlikely to be a problem.

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

to

==========
Drop [LegacyInterfaceTypeChecking] for the Presentation API

The change to the send() methods are unobservable, because the added
TypeError exceptions in the generated code are unreachable, as type
testing is first used to pick which send() to dispatch to.

The change to defaultRequest is observable, in that something like
`presentation.defaultRequest = {}` will now throw TypeError instead of
setting defaultRequest to null. Since this is a very new API, this is
very unlikely to be a problem.

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

[commit-bot: I haz the power, 2015-12-07 13:04:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-07 13:05:07]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] for the Presentation API

The change to the send() methods are unobservable, because the added
TypeError exceptions in the generated code are unreachable, as type
testing is first used to pick which send() to dispatch to.

The change to defaultRequest is observable, in that something like
`presentation.defaultRequest = {}` will now throw TypeError instead of
setting defaultRequest to null. Since this is a very new API, this is
very unlikely to be a problem.

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

to

==========
Drop [LegacyInterfaceTypeChecking] for the Presentation API

The change to the send() methods are unobservable, because the added
TypeError exceptions in the generated code are unreachable, as type
testing is first used to pick which send() to dispatch to.

The change to defaultRequest is observable, in that something like
`presentation.defaultRequest = {}` will now throw TypeError instead of
setting defaultRequest to null. Since this is a very new API, this is
very unlikely to be a problem.

BUG=561338

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

[commit-bot: I haz the power, 2015-12-07 13:05:07]

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