arni2033 Reporter
	Description • 8 years ago

>>>   My Info:   Win7_64, Nightly 47, 32bit, ID 20160222030212
STR:
1. Copy string  "example.org   "  in clipboard
2. Press Ctrl+T to open new tab
3. Paste the string copied in Step 1 in urlbar (Ctrl+V)
4.A) Click in searchbar
4.B) Click on the free place on page
4.C) Press Enter, then Escape to cancel loading (try this scenario with string "examexample.org   ")

AR:  "exa" in urlbar is gray, "mple.org" is black
ER:  The whole string in urlbar ("example.org") should be black

	arni2033 Reporter
	Comment 1 • 8 years ago

This is also reproducible with links like "example.org?asdf", "example.org#asdf".
I originally encountered this with "?asdf" type, but those STR were very specific.

This is regression from bug 1199289. Regression range:
> https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=30d61283668592adfaec7a00aeeabc9d32a688c2&tochange=417e00a78107e0e97f35a3e4df4819cd91f4781d

	:Gijs (he/him) Assignee
	Comment 2 • 8 years ago

(In reply to arni2033 from comment #1)
> This is also reproducible with links like "example.org?asdf",
> "example.org#asdf".
> I originally encountered this with "?asdf" type, but those STR were very
> specific.

I cannot reproduce these two issues on current Nightly, and believe they were fixed already. Can you provide more detail?

	:Gijs (he/him) Assignee
	Comment 3 • 8 years ago

(In reply to :Gijs Kruitbosch from comment #2)
> (In reply to arni2033 from comment #1)
> > This is also reproducible with links like "example.org?asdf",
> > "example.org#asdf".
> > I originally encountered this with "?asdf" type, but those STR were very
> > specific.
> 
> I cannot reproduce these two issues on current Nightly, and believe they
> were fixed already. Can you provide more detail?

Specifically, I think I fixed this in bug 1199601.

	:Gijs (he/him) Assignee
	Comment 4 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/36033/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/36033/

	arni2033 Reporter
	Comment 5 • 8 years ago

(In reply to :Gijs Kruitbosch from comment #2)
> > This is also reproducible with links like "example.org?asdf", "example.org#asdf".
> 
> I cannot reproduce these two issues on current Nightly, and believe they were fixed already.
Oh, that's correct. Those 2 cases were indeed fixed between DE 45 and Nightly 47.
I forgot to test them on Nightly.

	Jared Wein [:jaws] (please needinfo? me)
	Comment 6 • 8 years ago

Can you give an explanation as to how this change to the regex fixes the problem (for me and for posterity's sake)?

	:Gijs (he/him) Assignee
	Comment 7 • 8 years ago

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #6)
> Can you give an explanation as to how this change to the regex fixes the
> problem (for me and for posterity's sake)?

Replacing the '.' with \S ensures that we don't match whitespace characters as part of the domain/host (because whitespace can't be part of a domain). We need the \s* afterwards because otherwise the "$" 'end of string' marker doesn't match right after the domain (which now no longer includes the trailing whitespace) and we end up not highlighting the input at all.

	Jared Wein [:jaws] (please needinfo? me)
	Comment 8 • 8 years ago

Comment on attachment 8722442 [details]
MozReview Request: Bug 1250366 - fix whitespace issues with URL highlight code, r?jaws

https://reviewboard.mozilla.org/r/36033/#review32735

Thanks for the explanation and increased test coverage.

	Pulsebot
	Comment 9 • 8 years ago

https://hg.mozilla.org/integration/fx-team/rev/e02bcc28e90d

	Lina Butler [:lina]
	Comment 10 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/e02bcc28e90d

	Khalid Syfullah Zaman [:khalid32]
	Comment 11 • 8 years ago

I have reproduced this bug on Nightly 47.0a1 (2016-02-22) on ubuntu 14.04 LTS, 32 bit!

The bug's fix is now verified on Latest Developer Edition 47.0a2!

Build ID: 20160414004009
User Agent: Mozilla/5.0 (X11; Linux i686; rv:47.0) Gecko/20100101 Firefox/47.0