Set sequential focus navigation starting point on scrollIntoView()

Set sequential focus navigation starting point on scrollIntoView()

BUG=594613
Committed: https://crrev.com/0bb887b20c70582eeafad2a58ac1650b7159f2b6
Cr-Commit-Position: refs/heads/master@{#382522}

[kochi, 2016-03-18 10:51:26]

Description was changed from

==========
Set sequential focus navigation starting point on scrollIntoView()

BUG=594613
==========

to

==========
Set sequential focus navigation starting point on scrollIntoView()

BUG=594613
==========

[kochi, 2016-03-18 10:51:26]

kochi@chromium.org changed reviewers:
+ tkent@chromium.org

[kochi, 2016-03-18 10:52:35]

tkent-san,
Could you review?

[kochi, 2016-03-18 10:53:54]

On 2016/03/18 10:52:35, kochi wrote:
> tkent-san,
> Could you review?

BTW, I have no good idea how to test TextFinder.
Using browser_test?

[tkent, 2016-03-21 23:34:05]

The code change looks ok.

> BTW, I have no good idea how to test TextFinder.
> Using browser_test?

Does window.testRunner.findString() work well?

[kochi, 2016-03-22 05:54:27]

On 2016/03/21 23:34:05, tkent wrote:
> The code change looks ok.
> 
> > BTW, I have no good idea how to test TextFinder.
> > Using browser_test?
> 
> Does window.testRunner.findString() work well?

Thanks!  That's what I was looking for.

Updated the test.
Could you take another look?

[kochi, 2016-03-22 05:54:40]

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

[commit-bot: I haz the power, 2016-03-22 05:54:47]

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

[tkent, 2016-03-22 05:56:10]

lgtm

[kochi, 2016-03-22 06:02:01]

The CQ bit was unchecked by kochi@chromium.org

[kochi, 2016-03-22 06:02:01]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2016-03-22 06:02:09]

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

[commit-bot: I haz the power, 2016-03-22 07:04:25]

Description was changed from

==========
Set sequential focus navigation starting point on scrollIntoView()

BUG=594613
==========

to

==========
Set sequential focus navigation starting point on scrollIntoView()

BUG=594613
==========

[commit-bot: I haz the power, 2016-03-22 07:04:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-22 07:06:14]

Description was changed from

==========
Set sequential focus navigation starting point on scrollIntoView()

BUG=594613
==========

to

==========
Set sequential focus navigation starting point on scrollIntoView()

BUG=594613

Committed: https://crrev.com/0bb887b20c70582eeafad2a58ac1650b7159f2b6
Cr-Commit-Position: refs/heads/master@{#382522}
==========

[commit-bot: I haz the power, 2016-03-22 07:06:15]

Patchset 2 (id:??) landed as
https://crrev.com/0bb887b20c70582eeafad2a58ac1650b7159f2b6
Cr-Commit-Position: refs/heads/master@{#382522}