diff options
| author | finnur@google.com <finnur@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-08 20:29:43 +0000 |
|---|---|---|
| committer | finnur@google.com <finnur@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-08 20:29:43 +0000 |
| commit | e292fca9e47cba237bb5b2d885f900eddecd70c5 (patch) | |
| tree | 6b9a508844c0750364033bad598d24c350b3cdd4 /webkit | |
| parent | a2c257166578102f6874dace42dcbbfdb1441231 (diff) | |
| download | chromium_src-e292fca9e47cba237bb5b2d885f900eddecd70c5.zip chromium_src-e292fca9e47cba237bb5b2d885f900eddecd70c5.tar.gz chromium_src-e292fca9e47cba237bb5b2d885f900eddecd70c5.tar.bz2 | |
Fixing 5132: Find-in-page reports wrong ordinal when restarting search in sub-frame
ReportFindInPageSelection calls OrdinalOfFirstMatchForFrame internally, so we don't need to do it again at the call site.
Added a test to catch this in the future.
Review URL: http://codereview.chromium.org/13190
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@6532 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
| -rw-r--r-- | webkit/glue/webframe_impl.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/webkit/glue/webframe_impl.cc b/webkit/glue/webframe_impl.cc index 86fcb02..a3d003b 100644 --- a/webkit/glue/webframe_impl.cc +++ b/webkit/glue/webframe_impl.cc @@ -1116,7 +1116,7 @@ void WebFrameImpl::ScopeStringMatches(FindInPageRequest request, ReportFindInPageSelection( webkit_glue::FromIntRect( frame()->view()->convertToContainingWindow(result_bounds)), - OrdinalOfFirstMatchForFrame(this) + active_match_index_ + 1, + active_match_index_ + 1, request.request_id); #endif } |