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

Status: RESOLVED FIXED

Description

[rbs]

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 1

[rbs]

Created attachment 58072 [details] [diff] [review]
patch to clean, ready for r/sr

Comment 2

[Johnny Stenback (:jst)]

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

sr=jst

Comment 3

[Boris Zbarsky [:bzbarsky, bz on IRC]]

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?

Comment 4

[rbs]

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

[Boris Zbarsky [:bzbarsky, bz on IRC]]

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

r=bzbarsky.  works fine, less code == good.

Comment 6

[rbs]

Fixed.