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

VERIFIED FIXED

Status

Product:
Firefox for Android Graveyard
Component:
General
Status:
VERIFIED FIXED
Reported:
8 years ago
Modified:
8 years ago

People

(Reporter: vingtetun, Assigned: vingtetun)

Tracking

Version:
Trunk
Bug Flags:
in-litmus +

Details

(Whiteboard: [fennec-checkin-postb2])

Attachments

(1 attachment)

Patch
698 bytes, patch
mfinkle
: review+
Details | Diff | Splinter Review
(Assignee)

Description

•
8 years ago 
Created attachment 486612 [details] [diff] [review]
Patch

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]
Assignee: nobody → 21
 (Assignee)

Comment 1

•
8 years ago 
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
Last Resolved: 8 years ago
Resolution: --- → FIXED

Comment 2

•
8 years ago 
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)

Comment 3

•
8 years ago 
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.