Remove the special-case of viewsource in CSSLoaderImpl::LoadStyleLink()

RESOLVED FIXED

Status

()

Core
CSS Parsing and Computation
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: rbs, Assigned: rbs)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
Now that viewsource is handled very well by the security manager (bug 82495), 
there is no need for a special-case. Will attach a patch to clean up.
(Assignee)

Comment 1

16 years ago
Created attachment 58072 [details] [diff] [review]
patch to clean, ready for r/sr
Comment on attachment 58072 [details] [diff] [review]
patch to clean, ready for r/sr

sr=jst
Attachment #58072 - Flags: superreview+
Um..  if I read the patch in bug 82495 correctly, then view-source:http://foo
will end up having the same permissions as http://foo, no?  And then it won't be
able to load a stylesheet from resource://

Or am I missing something?
(Assignee)

Comment 4

16 years ago
I am not sure about all the intricacies in the security wolrd but I tested the
patch and it worked. Give a try at your end to see how it goes.
Comment on attachment 58072 [details] [diff] [review]
patch to clean, ready for r/sr

r=bzbarsky.  works fine, less code == good.
Attachment #58072 - Flags: review+
(Assignee)

Comment 6

16 years ago
Fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.