697105 - Permit spaces as a valid search query in an invalid URL

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Aaron Train [:aaronmt]]

STR:
1. "Toronto Ontario"
2. "The URL is not valid and cannot be loaded."

ER: https://www.google.com/q=toronto+ontario
AR: Dialog prompt

--
Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111025 Firefox/10.0a1
Samsung Galaxy SII (Android 2.3.4)

Comment 1

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

Attachment: patch

We load URLs directly in browserjs now, so we need to make sure those code paths are using 3RD_PARTY_FIXUP or nsIURIFixup.

This patch adds nsIURIFixup to Tab:Add and Tab:Load actions. I do wonder if we should show some kind of error dialog or error content page if the fixup fails.

Comment 2

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

Attachment: patch 2

Fixed a typo i last patch "URIFixer" -> "URIFixup"

Comment 3

[Doug Turner (:dougt)]

Comment on attachment 569601 [details] [diff] [review]
patch 2

suppose URIFixup returns null.  we silently fail.  Is it better to just pass whatever we got to loadURI() or addTab()?

createFixupURI(aData, 1);

to:

createFixupURI(aData, Ci.nsIURIFixup.FIXUP_FLAG_ALLOW_KEYWORD_LOOKUP);

or

createFixupURI(aData, 1 /* FIXUP_FLAG_ALLOW_KEYWORD_LOOKUP */ );

or similar

Comment 4

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

(In reply to Doug Turner (:dougt) from comment #3)
> Comment on attachment 569601 [details] [diff] [review] [diff] [details] [review]
> patch 2
> 
> suppose URIFixup returns null.  we silently fail.  Is it better to just pass
> whatever we got to loadURI() or addTab()?

Done. We use the URI if not null, else we pass the original data

> createFixupURI(aData, Ci.nsIURIFixup.FIXUP_FLAG_ALLOW_KEYWORD_LOOKUP);

Done

Comment 5

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

https://hg.mozilla.org/projects/birch/rev/186015786b5a

Comment 6

[Aaron Train [:aaronmt]]

Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111027 Firefox/10.0a1 Fennec/10.0a1
Samsung Nexus S