resource: URL's do not work in about page

VERIFIED WORKSFORME

Status

()

Core
Networking
P3
normal
VERIFIED WORKSFORME
18 years ago
18 years ago

People

(Reporter: Chris McAfee, Assigned: Norris Boyd)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
There is a resource URL for the user license in the
about: page now, URL looks correct but the link
does nothing.  Non-resource URL's work, this one does not.
marking beta1, as user cannot visit the license yet.
(Reporter)

Updated

18 years ago
Group: netscapeconfidential?
Keywords: beta1
(Reporter)

Updated

18 years ago
Blocks: 10980
(Reporter)

Updated

18 years ago
No longer blocks: 10980
(Reporter)

Comment 1

18 years ago
checked in beta workaround, relative link works.
removing beta1 keyword.
Keywords: beta1

Updated

18 years ago
Target Milestone: M15
(Reporter)

Comment 2

18 years ago
We've worked-around the broken license link with a 
relative URL, resummarizing for the general case.
Summary: Commercial license link does not work → resource: URL's do not work in about page

Comment 3

18 years ago
Marking beta2.
Keywords: beta2

Comment 4

18 years ago
I don't understand this bug. The logo is a resource url 
(resource:/chrome/global/content/default/logo.gif) and it loads ok. I need a 
test case or something.
Assignee: warren → mcafee
(Reporter)

Comment 5

18 years ago
the act of clicking on a resource: link doesn't work.
you don't have a test case because we worked around it!
A test case would be to change the license URL to
a resource URL, I'll attach a patch.

(Reporter)

Comment 6

18 years ago
opening to non-netscape people.
attaching patch
Group: netscapeconfidential?
(Reporter)

Comment 7

18 years ago
Created attachment 6813 [details] [diff] [review]
adding a test resource: URL you can click on
(Reporter)

Comment 8

18 years ago
back to warren

Assignee: mcafee → warren

Comment 9

18 years ago
The securitymanager catches this link ... nsScriptSecurityManager.cpp line 508.

Comment 10

18 years ago
Off to Norris. What should the rules be about loading about: urls from other 
pages? From resource: pages, it should be ok, but I'm not sure what else.
Assignee: warren → norris

Comment 11

18 years ago
Norris is out this week... and we need to cut the M15 checkpoint.  This problem 
does not seem to be a stability problem, so I'm pushing this to M16.
Target Milestone: M15 → M16
(Assignee)

Comment 12

18 years ago
References to about: URIs are now allowed.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → WORKSFORME

Updated

18 years ago
Keywords: nsbeta2

Comment 13

18 years ago
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.