[e10s] Quick search from the address bar doesn't work

RESOLVED FIXED

Status

Firefox for Android Graveyard
General
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: chithraprabha, Unassigned)

Tracking

Dependency tree / graph

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.18) Gecko/2010021501 Ubuntu/9.04 (jaunty) Firefox/3.0.18
Build Identifier: 

Quick search from the address bar doesn't work with electrolysis.

Reproducible: Always

Steps to Reproduce:
Open fennec and type some search words in the address bar and click go 
button.


Actual Results:  
Nothing happens.

Expected Results:  
Search results page is displayed.

below  exception occurs ,when the go button is pressed, 

"""WARNING: malformed hostname: file /home/chithrp/E10s/electrolysis/netwerk/base/src/nsURLParsers.cpp, line 606
JavaScript error: , line 0: uncaught exception: [Exception... "Component returned failure code: 0x804b000a (NS_ERROR_MALFORMED_URI) [nsIURIFixup.createFixupURI]"  nsresult: "0x804b000a (NS_ERROR_MALFORMED_URI)"  location: "JS frame :: chrome://browser/content/browser-ui.js :: anonymous :: line 436"  data: no] """

Comment 1

8 years ago
I suspect this can be resolved the same way as bug 554271.
Depends on: 554271

Updated

8 years ago
Blocks: 516521
(Reporter)

Comment 2

8 years ago
Created attachment 447294 [details] [diff] [review]
Patch V.0

With this patch quick search works fine .
Attachment #447294 - Flags: review?(mark.finkle)
Comment on attachment 447294 [details] [diff] [review]
Patch V.0


>-    let uri = gURIFixup.createFixupURI(aURI, 0);
>+    let  fixupFlags = gURIFixup.FIXUP_FLAG_ALLOW_KEYWORD_LOOKUP;
>+    let uri = gURIFixup.createFixupURI(aURI, fixupFlags);
> 
>     var flags = Ci.nsIWebNavigation.LOAD_FLAGS_ALLOW_THIRD_PARTY_FIXUP;
>     getBrowser().loadURIWithFlags(uri.spec, flags, null, null);

nits: extra space in "let  fixupFlags"
      use Ci.nsIURIFixup.FIXUP_FLAG...
      change "var flags" to "let loadFlags"

I can make the nit changes when landing
Attachment #447294 - Flags: review?(mark.finkle) → review+
ushed to mobile-e10s:
http://hg.mozilla.org/users/pavlov_mozilla.com/mobile-e10s/rev/88f97721f8bb

This patch should be safe for mobile-browser too, and landing it there would remove it from the "merge pain".

mobile-browser patch coming up
pushed sync patch to m-b:
http://hg.mozilla.org/mobile-browser/rev/84ea374b8a00
Status: UNCONFIRMED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.