Closed
Bug 110417
Opened 23 years ago
Closed 23 years ago
Remove the special-case of viewsource in CSSLoaderImpl::LoadStyleLink()
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
People
(Reporter: rbs, Assigned: rbs)
Details
Attachments
(1 file)
1.67 KB,
patch
|
bzbarsky
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
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.
Comment 2•23 years ago
|
||
Comment on attachment 58072 [details] [diff] [review] patch to clean, ready for r/sr sr=jst
Attachment #58072 -
Flags: superreview+
Comment 3•23 years ago
|
||
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?
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 5•23 years ago
|
||
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+
Fixed.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•