Closed Bug 551516 Opened 10 years ago Closed 10 years ago

Search button submission uses the wrong loadURI

Categories

(Firefox for Android Graveyard :: General, defect)

x86
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
Search engine submission can pass along POST data. Current code uses browser.loadURI, which does not support POST data. browser.loadURIWithFlags does support POST data.

This patch fixes the problem.
Attachment #431673 - Flags: review?(webapps)
Comment on attachment 431673 [details] [diff] [review]
patch

Not even nit, but just as an aside: I don't think passing passing the NONE constant as a flag is necessary (null works for me).  It's probably a little clearer your way since it gives us information about what that positional argument is.
Attachment #431673 - Flags: review?(webapps) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/d5b24ba0dd28
Assignee: nobody → mark.finkle
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.