For a range surrounding a node, set selection index to 0.

For a range surrounding a node, set selection index to 0.

ChromeVox already tries setting selection to its range when navigating. In the past, we passed -1 for the index (aka anchor/focus offsets), which had no effect.

After this change, within content editables, we sync our current range for commands that move it such as next heading.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/bcac09c87ee222e44a9bc1806315f762fcb29c91
Cr-Commit-Position: refs/heads/master@{#402909}

[David Tseng, 2016-06-24 20:37:35]

Description was changed from

==========
For a range surrounding a node, set selection index to 0.
==========

to

==========
For a range surrounding a node, set selection index to 0.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-06-28 17:57:23]

Description was changed from

==========
For a range surrounding a node, set selection index to 0.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
For a range surrounding a node, set selection index to 0.

ChromeVox already tries setting selection to its range when navigating. In the
past, we passed -1 for the index (aka anchor/focus offsets), which had no
effect.

After this change, within content editables, we sync our current range for
commands that move it such as next heading.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-06-28 17:57:23]

dtseng@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dmazzoni, 2016-06-28 17:58:22]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2016-06-28 17:58:22]

lgtm

[commit-bot: I haz the power, 2016-06-28 17:59:21]

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, 2016-06-28 18:26:12]

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

[commit-bot: I haz the power, 2016-06-28 18:26:13]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[David Tseng, 2016-06-28 19:25:43]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-06-28 19:25:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2099833002/#ps20001
(title: "Update test")

[commit-bot: I haz the power, 2016-06-28 19:27:27]

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, 2016-06-28 19:53:49]

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

[commit-bot: I haz the power, 2016-06-28 19:53:50]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[David Tseng, 2016-06-29 17:34:35]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-06-29 17:37:25]

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

[David Tseng, 2016-06-29 17:56:05]

The CQ bit was unchecked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-06-29 17:56:51]

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

[commit-bot: I haz the power, 2016-06-29 17:56:52]

CQ bit was unchecked.

[David Tseng, 2016-06-29 18:08:45]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-06-29 18:08:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2099833002/#ps40001
(title: "Add simple test.")

[commit-bot: I haz the power, 2016-06-29 18:10:29]

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, 2016-06-29 19:51:12]

Description was changed from

==========
For a range surrounding a node, set selection index to 0.

ChromeVox already tries setting selection to its range when navigating. In the
past, we passed -1 for the index (aka anchor/focus offsets), which had no
effect.

After this change, within content editables, we sync our current range for
commands that move it such as next heading.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
For a range surrounding a node, set selection index to 0.

ChromeVox already tries setting selection to its range when navigating. In the
past, we passed -1 for the index (aka anchor/focus offsets), which had no
effect.

After this change, within content editables, we sync our current range for
commands that move it such as next heading.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-06-29 19:51:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-29 19:52:49]

Description was changed from

==========
For a range surrounding a node, set selection index to 0.

ChromeVox already tries setting selection to its range when navigating. In the
past, we passed -1 for the index (aka anchor/focus offsets), which had no
effect.

After this change, within content editables, we sync our current range for
commands that move it such as next heading.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
For a range surrounding a node, set selection index to 0.

ChromeVox already tries setting selection to its range when navigating. In the
past, we passed -1 for the index (aka anchor/focus offsets), which had no
effect.

After this change, within content editables, we sync our current range for
commands that move it such as next heading.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2016-06-29 19:52:51]

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

[commit-bot: I haz the power, 2016-06-29 19:52:52]

CQ bit was unchecked.