Last Comment Bug 675710 - test_URIs.js doesn't test protocol-relative URIs ("//site/foo/bar")
: test_URIs.js doesn't test protocol-relative URIs ("//site/foo/bar")
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Networking (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: ---
Assigned To: Randell Jesup [:jesup]
:
: Patrick McManus [:mcmanus]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-01 12:38 PDT by Randell Jesup [:jesup]
Modified: 2011-08-02 06:57 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
protocol-relative URI tests (1.05 KB, patch)
2011-08-01 12:41 PDT, Randell Jesup [:jesup]
bzbarsky: review+
Details | Diff | Splinter Review

Description Randell Jesup [:jesup] 2011-08-01 12:38:39 PDT
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
Comment 1 Randell Jesup [:jesup] 2011-08-01 12:41:30 PDT
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
Comment 2 Boris Zbarsky [:bz] (still a bit busy) 2011-08-01 21:02:23 PDT
Comment on attachment 549889 [details] [diff] [review]
protocol-relative URI tests

r=me
Comment 3 Randell Jesup [:jesup] 2011-08-02 06:57:41 PDT
Checked in as http://hg.mozilla.org/mozilla-central/rev/8fe19339d17e

Note You need to log in before you can comment on or make changes to this bug.