Closed Bug 607965 Opened 12 years ago Closed 12 years ago

Find text field state is not reset when the find helper is closed

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: vingtetun, Assigned: vingtetun)

Details

(Whiteboard: [fennec-checkin-postb2])

Attachments

(1 file)

Patch
698 bytes, patch
mfinkle
: review+
Details | Diff | Splinter Review
Attached patch Patch — — Splinter Review 
Steps to reproduce:
 * go to about:home
 * go to the site menu -> Find In Page
 * type the string "test" -> observe the field background color goes red
 * dismiss the find helper by tapping in the content
 * go to the site menu -> Find In Page

Actual result:
 * the find's texfield is still red

Expected result:
 * The find textfield have a normal background color
Attachment #486612 - Flags: review?(mark.finkle)
Attachment #486612 - Flags: review?(mark.finkle) → review+
Whiteboard: [fennec-checkin-postb2]
http://hg.mozilla.org/mobile-browser/rev/8ccacaeb1d9d

I've added a small tweak to reset the status also when the field is cleared
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
verified FIXED on builds:
Mozilla/5.0 (Maemo; Linux armv71; rv:2.0b8pre) Gecko/20101105 Namoroka/4.0b8pre Fennec/4.0b2pre

and

Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101105 Namoroka/4.0b8pre Fennec/4.0b2pre
Status: RESOLVED → VERIFIED
Flags: in-litmus?(mozaakash)
litmus testcase https://litmus.mozilla.org/show_test.cgi?id=13790 created to regression test this bug
Flags: in-litmus?(mozaakash) → in-litmus+
You need to log in before you can comment on or make changes to this bug.