selection: Do not run invalid sub-tests with unselectable Range.

selection: Do not run invalid sub-tests with unselectable endpoints.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if they are in roots different
from Selection's document. We should skip such endpoints.

This CL removes 25,200 invalid tests.

BUG=683066
Review-Url: https://codereview.chromium.org/2700673005
Cr-Commit-Position: refs/heads/master@{#451545}
Committed: https://chromium.googlesource.com/chromium/src/+/329ba0e15e8853ef13e600776f1b3da4f30b0235

[tkent, 2017-02-17 15:15:29]

Description was changed from

==========
selection: Do not generated invalid sub-tests with unselectable Range.

BUG=
==========

to

==========
selection: Do not run invalid sub-tests with unselectable Range.

BUG=
==========

[tkent, 2017-02-17 15:15:35]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-17 15:15:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2017-02-17 16:17:27]

Description was changed from

==========
selection: Do not run invalid sub-tests with unselectable Range.

BUG=
==========

to

==========
selection: Do not run invalid sub-tests with unselectable Range.

This CL removes 23,896 invalid tests.

BUG=
==========

[tkent, 2017-02-17 16:27:06]

Description was changed from

==========
selection: Do not run invalid sub-tests with unselectable Range.

This CL removes 23,896 invalid tests.

BUG=
==========

to

==========
selection: Do not run invalid sub-tests with unselectable Range.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if it is in a root different
from Selection's document. We should skip such endpoints.

This CL removes 23,896 invalid tests.

BUG=683066
==========

[tkent, 2017-02-17 16:28:24]

Description was changed from

==========
selection: Do not run invalid sub-tests with unselectable Range.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if it is in a root different
from Selection's document. We should skip such endpoints.

This CL removes 23,896 invalid tests.

BUG=683066
==========

to

==========
selection: Do not run invalid sub-tests with unselectable endpoints.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if they are in roots different
from Selection's document. We should skip such endpoints.

This CL removes 23,896 invalid tests.

BUG=683066
==========

[tkent, 2017-02-17 16:29:15]

tkent@chromium.org changed reviewers:
+ yoichio@chromium.org, yosin@chromium.org

[tkent, 2017-02-17 16:29:16]

yoichio@, yosin@, would you review this please?

[commit-bot: I haz the power, 2017-02-17 16:35:39]

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

[commit-bot: I haz the power, 2017-02-17 16:35:40]

Dry run: This issue passed the CQ dry run.

[tkent, 2017-02-19 23:41:14]

Description was changed from

==========
selection: Do not run invalid sub-tests with unselectable endpoints.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if they are in roots different
from Selection's document. We should skip such endpoints.

This CL removes 23,896 invalid tests.

BUG=683066
==========

to

==========
selection: Do not run invalid sub-tests with unselectable endpoints.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if they are in roots different
from Selection's document. We should skip such endpoints.

This CL removes 25,200 invalid tests.

BUG=683066
==========

[tkent, 2017-02-19 23:41:17]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-19 23:41:35]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-20 00:48:35]

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

[commit-bot: I haz the power, 2017-02-20 00:48:36]

Dry run: This issue passed the CQ dry run.

[yoichio, 2017-02-20 01:58:33]

lgtm

[tkent, 2017-02-20 02:16:56]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2017-02-20 02:17:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-20 02:24:03]

CQ is committing da patch.

Bot data: {"patchset_id": 10001, "attempt_start_ts": 1487557016717000,
"parent_rev": "a8725caafe812944aa590f27e8711eab810bf1a1", "commit_rev":
"329ba0e15e8853ef13e600776f1b3da4f30b0235"}

[commit-bot: I haz the power, 2017-02-20 02:25:04]

Description was changed from

==========
selection: Do not run invalid sub-tests with unselectable endpoints.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if they are in roots different
from Selection's document. We should skip such endpoints.

This CL removes 25,200 invalid tests.

BUG=683066
==========

to

==========
selection: Do not run invalid sub-tests with unselectable endpoints.

* addRange.js:
It assumes any Range can be added to Selection. However the
latest draft defines that addRange() rejects Ranges in a root
different from Selection's document.
Some tests after addRange() with such Ranges were invalid.

https://w3c.github.io/selection-api/#dom-selection-addrange
> The method must set the context object's range to range by a
> strong reference (not by making a copy) if the root ([DOM4]) of
> the range's boundary points are the document associated with
> context object. Otherwise, this method must do nothing.

* exntend.js
extend() requires rangeCount>=1. setSelectionForwards() and
setSelectionBackwards() are assumed to ensure it. However, these
function can't set specified endpoints if they are in roots different
from Selection's document. We should skip such endpoints.

This CL removes 25,200 invalid tests.

BUG=683066

Review-Url: https://codereview.chromium.org/2700673005
Cr-Commit-Position: refs/heads/master@{#451545}
Committed:
https://chromium.googlesource.com/chromium/src/+/329ba0e15e8853ef13e600776f1b...
==========

[commit-bot: I haz the power, 2017-02-20 02:25:06]

Committed patchset #2 (id:10001) as
https://chromium.googlesource.com/chromium/src/+/329ba0e15e8853ef13e600776f1b...