597203 - Find in Page buttons are not disabled when only one word found on page

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Aakash Desai [:aakashd]]

Build id:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b6pre) Gecko/20100916 Namoroka/4.0b7pre Fennec/2.0b1pre

Steps to Reproduce:
1. Go to http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android-r7/
2. Click on the favicon
3. Click on "Find in Page"
4. Type in "mobile" and press Enter

Actual Results:
The up/down arrows on the find in page bar are clickable

Expected Results:
The up/down arrows on the find in page bar are disabled

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Do we even know how many words are found?

Comment 2

[Aakash Desai [:aakashd]]

What do you mean? It should only be one since there aren't any other instances of "mobile" anywhere else on that page.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

I don't see how this can be implemented using the nsITypeAheadFind service:
http://mxr.mozilla.org/mozilla-central/source/toolkit/components/typeaheadfind/public/nsITypeAheadFind.idl

There appear to be no way to determine how many instances of a match occur on a page. In fact, desktop Firefox doesn't seem to disable the next/prev buttons either.

Until a better core component exists, this won't happen.