Closed Bug 995347 Opened 8 years ago Closed 8 years ago

Entering "h" in URL bar shows "http:/" or "https:/" as autocompletion


(Firefox for Android Graveyard :: General, defect)

Not set


(firefox28 affected, firefox29 affected, firefox30 affected, firefox31 affected, fennec+)

Firefox 31
Tracking Status
firefox28 --- affected
firefox29 --- affected
firefox30 --- affected
firefox31 --- affected
fennec + ---


(Reporter: bnicholson, Assigned: sahai.anuj16)



(2 files, 1 obsolete file)

Attached image screenshot
Rather than completing a domain, I just see "http:/" or "https:/" when entering "h", which isn't very helpful. See screenshot.
Can reproduce in release, so doesn't look like a regression.
Whiteboard: [good first bug?]
tracking-fennec: --- → ?
Hi, I would like to take this up if it's okay.
Hi, I'm a bit unclear on what the expected behavior is in this case. For example, if the suggestion is, say,, should typing h just fill in the domain ( or the entire url (

Never mind, I found that autocomplete normally fills in segment by segment, so, it should be in the above case.
Assignee: nobody → sahai.anuj16
Attachment #8407776 - Flags: review?(rnewman)
Comment on attachment 8407776 [details] [diff] [review]
Allow domain auto completion when entering "h" in url bar

I don't have time to tackle this right now; Brian?
Attachment #8407776 - Flags: review?(rnewman) → review?(bnicholson)
Comment on attachment 8407776 [details] [diff] [review]
Allow domain auto completion when entering "h" in url bar

Review of attachment 8407776 [details] [diff] [review]:

Thanks! This could technically break for any results that have a "/" before the 9th character and don't begin with "http://" or "https://", but I can't think of any scenarios where this would actually happen.

::: mobile/android/base/home/
@@ +422,5 @@
>              // Does the completion match against the whole URL? This will match
>              // about: pages, as well as user input including "http://...".
>              if (url.startsWith(searchTerm)) {
> +                return uriSubstringUpToMatchedPath(url, 0,
> +                        searchLength > HTTPS_PREFIX_LENGTH ? searchLength : HTTPS_PREFIX_LENGTH);

Nit: Parens around (searchLength > HTTPS_PREFIX_LENGTH).
Attachment #8407776 - Flags: review?(bnicholson) → review+
This would only be applicable for about:.., http://..., ftp://.. and so on. If a slash (/) appears before the 9th character (can't imagine a real scenario), it would still auto complete till the segment that has the 9th character.
Attachment #8407776 - Attachment is obsolete: true
Attachment #8408280 - Flags: review?(bnicholson)
tracking-fennec: ? → +
Comment on attachment 8408280 [details] [diff] [review]
Allow domain auto completion when entering "h" in url bar [Added parentheses]

Looks good, thanks again!

Are you familiar with the checkin-needed flag? If not: this is the way to get your patch landed after it's ready to go. You can add checkin-needed as a keyword in this bug (top of the page), and one of our volunteers will land it for you.
Attachment #8408280 - Flags: review?(bnicholson) → review+
Hey Brian, thanks a lot, I was hunting through the docs to see what the next step is supposed to be. :)
Keywords: checkin-needed
Keywords: checkin-needed
Whiteboard: [good first bug?] → [good first bug?][fixed-in-fx-team]
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug?][fixed-in-fx-team] → [good first bug?]
Target Milestone: --- → Firefox 31
Whiteboard: [good first bug?]
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.