Revert of Fix find-in-page re-scope across frame boundaries.

Revert of Fix find-in-page re-scope across frame boundaries. (patchset #4 id:120001 of https://codereview.chromium.org/2186113002/ )

Reason for revert:
Multiple bots are timing out on FindRequestManagerTest.RemoveFrame/1 since this patch landed. No one else has touched FindRequestManagerTests recently.

Original issue's description:
> Fix find-in-page re-scope across frame boundaries.
>
> There is a bug that can cause a frame to be re-scoped (all matches searched again) when issuing a find next request across a frame boundary, such as from the last match in a frame, which might need to wrap back around to the first match in the same frame. If there are not many matches in that frame, this is not really noticeable except for the ticks in the tickbar sometimes disappearing for a single frame. However, on a page with a huge number of results, this can dramatically slow down the find request and you can clearly see all of the ticks in the tickbar regenerate.
>
> This patch adjusts the conditions under which a frame is determined to need scoping in WebLocalFrameImpl::requestFind(), in order to not re-scope across frame boundaries but continue to scope in all the cases where it IS needed.
>
> BUG=618937, 627799
>
> Committed: https://crrev.com/b6f12c557fb9b1468ba6f6da5f55cc7cefc045b0
> Cr-Commit-Position: refs/heads/master@{#412025}

TBR=esprehn@chromium.org,paulmeyer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=618937, 627799
Committed: https://crrev.com/9f9426efd4f57e650ffc87da7df09766be962875
Cr-Commit-Position: refs/heads/master@{#412068}

[Bret, 2016-08-15 22:13:45]

The CQ bit was checked by bsep@chromium.org

[Bret, 2016-08-15 22:13:45]

Created Revert of Fix find-in-page re-scope across frame boundaries.

[commit-bot: I haz the power, 2016-08-15 22:14:08]

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-08-15 22:14:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-15 22:17:45]

Description was changed from

==========
Revert of Fix find-in-page re-scope across frame boundaries. (patchset #4
id:120001 of https://codereview.chromium.org/2186113002/ )

Reason for revert:
Multiple bots are timing out on FindRequestManagerTest.RemoveFrame/1 since this
patch landed. No one else has touched FindRequestManagerTests recently.

Original issue's description:
> Fix find-in-page re-scope across frame boundaries.
> 
> There is a bug that can cause a frame to be re-scoped (all matches searched
again) when issuing a find next request across a frame boundary, such as from
the last match in a frame, which might need to wrap back around to the first
match in the same frame. If there are not many matches in that frame, this is
not really noticeable except for the ticks in the tickbar sometimes disappearing
for a single frame. However, on a page with a huge number of results, this can
dramatically slow down the find request and you can clearly see all of the ticks
in the tickbar regenerate.
> 
> This patch adjusts the conditions under which a frame is determined to need
scoping in WebLocalFrameImpl::requestFind(), in order to not re-scope across
frame boundaries but continue to scope in all the cases where it IS needed.
> 
> BUG=618937,627799
> 
> Committed: https://crrev.com/b6f12c557fb9b1468ba6f6da5f55cc7cefc045b0
> Cr-Commit-Position: refs/heads/master@{#412025}

TBR=esprehn@chromium.org,paulmeyer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=618937,627799
==========

to

==========
Revert of Fix find-in-page re-scope across frame boundaries. (patchset #4
id:120001 of https://codereview.chromium.org/2186113002/ )

Reason for revert:
Multiple bots are timing out on FindRequestManagerTest.RemoveFrame/1 since this
patch landed. No one else has touched FindRequestManagerTests recently.

Original issue's description:
> Fix find-in-page re-scope across frame boundaries.
>
> There is a bug that can cause a frame to be re-scoped (all matches searched
again) when issuing a find next request across a frame boundary, such as from
the last match in a frame, which might need to wrap back around to the first
match in the same frame. If there are not many matches in that frame, this is
not really noticeable except for the ticks in the tickbar sometimes disappearing
for a single frame. However, on a page with a huge number of results, this can
dramatically slow down the find request and you can clearly see all of the ticks
in the tickbar regenerate.
>
> This patch adjusts the conditions under which a frame is determined to need
scoping in WebLocalFrameImpl::requestFind(), in order to not re-scope across
frame boundaries but continue to scope in all the cases where it IS needed.
>
> BUG=618937,627799
>
> Committed: https://crrev.com/b6f12c557fb9b1468ba6f6da5f55cc7cefc045b0
> Cr-Commit-Position: refs/heads/master@{#412025}

TBR=esprehn@chromium.org,paulmeyer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=618937,627799

Committed: https://crrev.com/9f9426efd4f57e650ffc87da7df09766be962875
Cr-Commit-Position: refs/heads/master@{#412068}
==========

[commit-bot: I haz the power, 2016-08-15 22:17:46]

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