Closed Bug 646444 Opened 9 years ago Closed 9 years ago

Add search engine not present in page's favicon.

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ioana.chiorean, Assigned: vingtetun)

Details

Attachments

(1 file)

Build id : Mozilla/5.0 (Android;Linux armv7l;rv:2.2a1pre)Gecko/20110330
Firefox/4.2a1pre Fennec /4.01a1pre
Device: Samsung Galaxy S Captivate
OS: Android 2.1 update 1

Steps to reproduce:
1. Go to AMO or www.bugzilla.mozilla.org
2. Tap the favicon 

Actual result:
- option "Add search enginer" is not present in the pop up menu

Expected result: 
- option "Add search enginer" is present in the pop up menu
Attached patch PatchSplinter Review
This is likely a regression from http://hg.mozilla.org/mozilla-central/rev/c1f5c784a38b , i have a patch for that but I think we need to ensure the failure is expected first.
Bug 582717 made regular expressions not callable, so this is expected.
Comment on attachment 522991 [details] [diff] [review]
Patch

(In reply to comment #2)
> Bug 582717 made regular expressions not callable, so this is expected.

Thanks for the info.
Attachment #522991 - Flags: review?(mark.finkle)
Attachment #522991 - Flags: review?(mark.finkle) → review+
http://hg.mozilla.org/mobile-browser/rev/bba3e2c500df
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Verified on build: Mozilla /5.0 (Android;Linux armv7l;rv:2.2a1pre) Gecko/20110331 Firefox/4.2a1pre Fennec /4.1a1pre 
Device: HTC Desire
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.