Make WebLocalFrameImpl::Load() to reset active match

Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|.

This is caused by processing
|WebLocalFrame::StopFInding()| with |kStopFindActionKeepSelection| from
|FindRequestManager| in browser process is happend after completion of
loading document.


BUG=719880
TEST=run_webkit_unit_tests All/ParameterizedWebFrameTest.FindInPageStopFindActionKeepSelectionInAnotherDocument/*

Review-Url: https://codereview.chromium.org/2942623003
Cr-Commit-Position: refs/heads/master@{#479636}
Committed: https://chromium.googlesource.com/chromium/src/+/7920530b27ad7c75a12f02b410ba8dbf923a2df8

[yosin_UTC9, 2017-06-15 04:38:16]

Description was changed from

==========
2017-06-15T13:35:04

BUG=719880

2017-06-15T13:34:10
==========

to

==========
Make WebLocalFrameImpl::SetFindEndstateFocusAndSelection() not to use active
match for another document

BUG=719880

2017-06-15T13:34:10
==========

[yosin_UTC9, 2017-06-15 04:38:20]

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

[commit-bot: I haz the power, 2017-06-15 04:38:49]

Dry run: CQ is trying da patch.

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

[yosin_UTC9, 2017-06-15 05:15:17]

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

[commit-bot: I haz the power, 2017-06-15 05:15:32]

Dry run: CQ is trying da patch.

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

[yosin_UTC9, 2017-06-15 05:18:51]

Description was changed from

==========
Make WebLocalFrameImpl::SetFindEndstateFocusAndSelection() not to use active
match for another document

BUG=719880

2017-06-15T13:34:10
==========

to

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|. This is caused by calling |WebLocalFrame::StopFInding()|
with |kStopFindActionKeepSelection| after loading document.

BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageSetFindEndstateFocusAndSelection/*
==========

[yosin_UTC9, 2017-06-15 05:22:32]

Description was changed from

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|. This is caused by calling |WebLocalFrame::StopFInding()|
with |kStopFindActionKeepSelection| after loading document.

BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageSetFindEndstateFocusAndSelection/*
==========

to

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|.

This is caused by processing
|WebLocalFrame::StopFInding()| with |kStopFindActionKeepSelection| from
|FindRequestManager| in browser process is happend after completion of
loading document.


BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageSetFindEndstateFocusAndSelection/*
==========

[yosin_UTC9, 2017-06-15 05:26:38]

Description was changed from

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|.

This is caused by processing
|WebLocalFrame::StopFInding()| with |kStopFindActionKeepSelection| from
|FindRequestManager| in browser process is happend after completion of
loading document.


BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageSetFindEndstateFocusAndSelection/*
==========

to

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|.

This is caused by processing
|WebLocalFrame::StopFInding()| with |kStopFindActionKeepSelection| from
|FindRequestManager| in browser process is happend after completion of
loading document.


BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageStopFindActionKeepSelectionInAnotherDocument/*
==========

[yosin_UTC9, 2017-06-15 05:27:03]

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

[yosin_UTC9, 2017-06-15 05:27:09]

Patchset #2 (id:20001) has been deleted

[yosin_UTC9, 2017-06-15 05:27:14]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-06-15 05:27:15]

Dry run: CQ is trying da patch.

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

[yosin_UTC9, 2017-06-15 06:39:15]

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

[yosin_UTC9, 2017-06-15 06:39:15]

PTAL

[tkent, 2017-06-15 07:14:10]

lgtm

[yosin_UTC9, 2017-06-15 07:29:34]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2017-06-15 07:29:36]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-06-15 07:29:41]

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-06-15 08:14:44]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497511776834030,
"parent_rev": "772e2f76cdab92e73a50192f07b2ea7036c06b79", "commit_rev":
"9ee03af276064894770f01792e23bc66ea97976e"}

[commit-bot: I haz the power, 2017-06-15 08:14:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497511776834030,
"parent_rev": "7cb90dcac57732686408a127941865df2aa2b577", "commit_rev":
"7920530b27ad7c75a12f02b410ba8dbf923a2df8"}

[commit-bot: I haz the power, 2017-06-15 08:15:09]

Description was changed from

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|.

This is caused by processing
|WebLocalFrame::StopFInding()| with |kStopFindActionKeepSelection| from
|FindRequestManager| in browser process is happend after completion of
loading document.


BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageStopFindActionKeepSelectionInAnotherDocument/*
==========

to

==========
Make WebLocalFrameImpl::Load() to reset active match

This patch changes |WebLocalFrameImpl::Load()| to reset active match, which is
represented by |Range|, to avoid setting selection with document not hosted
by |WebLocalFrameImpl|.

This is caused by processing
|WebLocalFrame::StopFInding()| with |kStopFindActionKeepSelection| from
|FindRequestManager| in browser process is happend after completion of
loading document.


BUG=719880
TEST=run_webkit_unit_tests
All/ParameterizedWebFrameTest.FindInPageStopFindActionKeepSelectionInAnotherDocument/*

Review-Url: https://codereview.chromium.org/2942623003
Cr-Commit-Position: refs/heads/master@{#479636}
Committed:
https://chromium.googlesource.com/chromium/src/+/7920530b27ad7c75a12f02b410ba...
==========

[commit-bot: I haz the power, 2017-06-15 08:15:10]

Committed patchset #1 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/7920530b27ad7c75a12f02b410ba...