Closed
Bug 110417
Opened 24 years ago
Closed 24 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•24 years ago
|
||
Comment on attachment 58072 [details] [diff] [review]
patch to clean, ready for r/sr
sr=jst
Attachment #58072 -
Flags: superreview+
![]() |
||
Comment 3•24 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•24 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: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•