test_URIs.js doesn't test protocol-relative URIs ("//site/foo/bar")

RESOLVED FIXED

Status

()

Core
Networking
--
trivial
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
We handle protocol-relative URIs, but they're not in the test suite
See RFC 3986 section 4.2:
http://tools.ietf.org/html/rfc3986#section-4.2

Follow-on to bug 622981
(Assignee)

Updated

6 years ago
Summary: test_URIs.js doesn't test protocol-relative URIs ("//site/fo/bar") → test_URIs.js doesn't test protocol-relative URIs ("//site/foo/bar")
(Assignee)

Comment 1

6 years ago
Created attachment 549889 [details] [diff] [review]
protocol-relative URI tests

The tests pass, but this adds coverage in case anyone breaks it.

No affect on normal builds, just on test runs
Attachment #549889 - Flags: review?(bzbarsky)
Comment on attachment 549889 [details] [diff] [review]
protocol-relative URI tests

r=me
Attachment #549889 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 3

6 years ago
Checked in as http://hg.mozilla.org/mozilla-central/rev/8fe19339d17e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.