Replace string.contains with string.endsWith in formatValue

RESOLVED FIXED in Firefox 18

Status

()

Firefox
Location Bar
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jaws, Assigned: jaws)

Tracking

Trunk
Firefox 18
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 665032 [details] [diff] [review]
Patch

From https://bugzilla.mozilla.org/show_bug.cgi?id=781588#c41,
> Comment on attachment 663537 [details] [diff] [review]
> Patch v6
> 
> >+              if (!domain.contains(baseDomain)) {
> Should this be endsWith?

I tested this out and the browser_urlHighlight.js test still passes.
Attachment #665032 - Flags: review?(felipc)

Updated

5 years ago
Attachment #665032 - Flags: review?(felipc) → review+

Updated

5 years ago
Component: Tabbed Browser → Location Bar
Thanks for the quick review Dao!

https://hg.mozilla.org/integration/mozilla-inbound/rev/be63bddab58b
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/be63bddab58b
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
You need to log in before you can comment on or make changes to this bug.