Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

DOMSelection::addRange is only called from generated bindings, so the
assert will hold.

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

[philipj_slow, 2015-12-09 10:32:06]

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

[commit-bot: I haz the power, 2015-12-09 10:32:54]

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

[philipj_slow, 2015-12-09 12:54:34]

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

[philipj_slow, 2015-12-09 12:54:34]

PTAL

[haraken, 2015-12-09 13:55:57]

LGTM

[philipj_slow, 2015-12-09 14:28:40]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

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

to

==========
Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

DOMSelection::addRange is only called from generated bindings, so the
assert will hold.

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

[philipj_slow, 2015-12-09 14:28:49]

The CQ bit was unchecked by philipj@opera.com

[philipj_slow, 2015-12-09 14:28:51]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-12-09 14:29:17]

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

[commit-bot: I haz the power, 2015-12-09 16:07:09]

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

[commit-bot: I haz the power, 2015-12-09 16:07:10]

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_...)

[philipj_slow, 2015-12-09 17:47:24]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-12-09 17:48:39]

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

[commit-bot: I haz the power, 2015-12-09 20:23:44]

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

[commit-bot: I haz the power, 2015-12-09 20:23:45]

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_...)

[philipj_slow, 2015-12-09 21:19:56]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-12-09 21:21:25]

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

[commit-bot: I haz the power, 2015-12-10 00:01:01]

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

[commit-bot: I haz the power, 2015-12-10 00:01:02]

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_...)

[brucedawson, 2015-12-10 01:20:15]

On 2015/12/10 00:01:02, commit-bot: I haz the power wrote:
> 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_...)

Failing due to https://code.google.com/p/chromium/issues/detail?id=567377 - two
hour job timeout

[philipj_slow, 2015-12-10 08:10:41]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-12-10 08:11:35]

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

[commit-bot: I haz the power, 2015-12-10 09:19:26]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

DOMSelection::addRange is only called from generated bindings, so the
assert will hold.

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

to

==========
Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

DOMSelection::addRange is only called from generated bindings, so the
assert will hold.

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

[commit-bot: I haz the power, 2015-12-10 09:19:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-10 09:20:34]

Description was changed from

==========
Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

DOMSelection::addRange is only called from generated bindings, so the
assert will hold.

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

to

==========
Drop [LegacyInterfaceTypeChecking] for Selection.prototype.addRange

This already throws in Firefox and IE11 when the argument is null:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3781

DOMSelection::addRange is only called from generated bindings, so the
assert will hold.

BUG=561338

Committed: https://crrev.com/22f68ef35ec7ad8590992913ef8733ceb9c911d3
Cr-Commit-Position: refs/heads/master@{#364322}
==========

[commit-bot: I haz the power, 2015-12-10 09:20:35]

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