195625 - browser.docShell.allowImages = false; also hides scrollbar images!

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Embedding: APIs, defect)

Description

[HJ]

Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:1.3b; MultiZilla v1.3.1 (e))
Gecko/20030302

Steps to reproduce:
- set browser.docShell.allowImages = false;
- take a look at your scrollbar images

Current result : all chrome://global/skin//scrollbar/*.gif are gone
Expected Result: display all chrome: related images

Now, take a look at this:
http://lxr.mozilla.org/seamonkey/source/embedding/browser/webBrowser/nsWebBrowserContentPolicy.cpp#68

I think that nsIURI *contentLocation should be used to check for a chrome:
protocal first, and not simply return shell->GetAllowImages(shouldLoad);

Comment 1

[HJ]

Oh, just for the record: 
prefs.setIntPref("network.image.imageBehavior", [1/2]); does not expose this bug!

Comment 2

[Josh Birnbaum]

I'm being blind...  is Moz 1.3b supposed to have a
"browser.docShell.allowImages" pref?  I did an lxr search and came up empty.

Comment 3

[HJ]

Josh, it's not a pref, just look at this:

http://lxr.mozilla.org/seamonkey/source/docshell/base/nsIDocShell.idl#216

Comment 4

[Boris Zbarsky [:bzbarsky]]

Embedding apis.

Comment 5

[HJ]

I am nominating this bug for mozilla 1.6b because this bug still assists and
kills theme support. It should be easy to fix, because all it needs is a check
for chrome:

Comment 6

[HJ]

s/assists/exists

Comment 7

[Andrew Schultz]

HJ: only drivers can set (+) blocking flags.  you can request (?) them.

Comment 8

[HJ]

Hm, well I *can* set these flags, as you've noticed, but I guess I should not be
allowed to do so by bugzilla rights, however, thanks for the tip.

Comment 9

[Benjamin Smedberg]

Marking a bunch of bugs in the "Embedding: APIs" component INCOMPLETE in preparation to archive that component. If I have done this incorrectly, please reopen the bugs and move them to a more correct component as we don't have "embedding" APIs any more.