689599 - search drop-down does not work while on an about: page

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Tim Taubert [:ttaubert] (inactive)]

STR:

1) Ensure Fennec is closed.
2) Start Fennec (about:fennec is shown)
3) Type a search term into the url bar
4) Click the magnifier icon at the very right
5) Select "Google"

Expected Result:

Google search for the given search term is done.

Actual Result:

A blank page is shown.

Comment 1

[Matt Brubeck (:mbrubeck)]

I can reproduce this in desktop Fennec.  It looks like this happens any time you use the search engine menu while a local tab ("about:" or "chrome:" page) is selected.

Comment 2

[Matt Brubeck (:mbrubeck)]

Regression from bug 685440.

Comment 3

[Matt Brubeck (:mbrubeck)]

Attachment: patch

The bug is caused by the inadequate check for postData in loadURI, which threw an exception when aParams.postData was null.

The other changes are just minor fixes to related code.

Comment 4

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

We'll want this in Fx9 (Aurora)

Comment 5

[Matt Brubeck (:mbrubeck)]

Comment on attachment 562794 [details] [diff] [review]
patch

Pushed to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/45e745c780d7

Requesting approval-mozilla-aurora for Firefox 9.  This fixes a regression introduced in Firefox 9 that breaks searching from the address bar in some common situations.  The fix is mobile-only and very safe.  It just adds a null check before accessing a property.

Comment 6

[Matt Brubeck (:mbrubeck)]

Pushed to Aurora for Firefox 9:
https://hg.mozilla.org/releases/mozilla-aurora/rev/2b4f99e468aa

Comment 7

[Marco Bonardo [:mak]]

https://hg.mozilla.org/mozilla-central/rev/45e745c780d7

Comment 8

[Cristian Nicolae (:xti)]

This issue is not reproducible anymore on the latest Aurora build, but I was able to reproduce it on the latest Nightly build.

--
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20110928
Firefox/9.0a2 Fennec/9.0a2
Device: Samsung Galaxy S
OS: Android 2.2

Mozilla/5.0 (Android;Linux armv7l;rv:10.0a1)Gecko/20110928
Firefox/10.0a1 Fennec/10.0a1
Device: Samsung Galaxy S
OS: Android 2.2

Comment 9

[Aaron Train [:aaronmt]]

(In reply to Cristian Nicolae (:xti) from comment #8)
> This issue is not reproducible anymore on the latest Aurora build, but I was
> able to reproduce it on the latest Nightly build.
> 
> --
> Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20110928
> Firefox/9.0a2 Fennec/9.0a2
> Device: Samsung Galaxy S
> OS: Android 2.2
> 
> Mozilla/5.0 (Android;Linux armv7l;rv:10.0a1)Gecko/20110928
> Firefox/10.0a1 Fennec/10.0a1
> Device: Samsung Galaxy S
> OS: Android 2.2

09/28's Nightly was built off of 7f4867717226, this landed with 45e745c780d7 which didn't make the Nightly build. Check this again tomorrow.

Comment 10

[Camelia Urian]

Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20110929
Firefox/9.0a2 Fennec/9.0a2
Device: HTC Desire Z
OS: Android 2.3

Mozilla/5.0 (Android;Linux armv7l;rv:10.0a1)Gecko/20110929
Firefox/10.0a1 Fennec/10.0a1
Device: HTC Desire Z
OS: Android 2.3

Verified on both Nightly and Aurora, and issue is not reproducing.

Comment 11

[Martijn Wargers (dead)]

With the steps from comment 0, I cannot reproduce this issue on the LG Optimus Black, using Android OS 2.2.2, using the 10b1 build from:
http://ftp.mozilla.org/pub/mozilla.org/mobile/candidates/10.0b1-candidates/build1/unsigned/
So marking status-firefox10-verified.