The default bug view has changed. See this FAQ.

Location Bar doesn't highlight domains with ports

RESOLVED FIXED in Firefox 10

Status

()

Firefox
Location Bar
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Eric Sh., Assigned: dao)

Tracking

({regression})

Trunk
Firefox 10
regression
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 558911 [details]
Example

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0
Build ID: 20110902161802

Steps to reproduce:

Enter a domain with a port such as localhost:80/path/to/something


Actual results:

All of the text in the location bar is black


Expected results:

The domain part and the port should be highlighted
(Reporter)

Updated

6 years ago
(Reporter)

Updated

6 years ago
(Assignee)

Updated

6 years ago
Blocks: 451833
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: x86 → All
Version: 7 Branch → Trunk
(Reporter)

Comment 1

6 years ago
Anyone has an idea where could I find this code and maybe try and fix this myself?
http://mxr.mozilla.org/mozilla-central/source/browser/base/content/urlbarBindings.xml#195
(Assignee)

Updated

6 years ago
Blocks: 665580
Keywords: regression
(Assignee)

Comment 3

6 years ago
Created attachment 565173 [details] [diff] [review]
patch: don't parse leading "foo:" as a protocol when followed by a number
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #565173 - Flags: review?(gavin.sharp)
Attachment #565173 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/094ec090a211
Flags: in-testsuite+
Target Milestone: --- → Firefox 10
https://hg.mozilla.org/mozilla-central/rev/094ec090a211
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Comment 6

6 years ago
Thank you confirmed as fixed in latest nightly.
You need to log in before you can comment on or make changes to this bug.