30322 - 2nd visit to mixed site shows locked lock

Status: VERIFIED DUPLICATE of bug 33762

(Core Graveyard :: Security: UI, defect, P3)

Description

[John Unruh]

1.) Visit https://junruh.mcom.com/mix.html. Note the mixed content lock.
2.) Click on Lock Testing #6 - the same site.

What is expected: That the lock icon would have a slash in front of it.
What happens: The lock icon is locked.

Comment 1

[Doug Turner (:dougt)]

This is caused by nsIDocumentLoaderObserver not firing on reload (32229).

Comment 2

[Doug Turner (:dougt)]

m17

Comment 3

[Phil Peterson]

nsbeta2+

Comment 4

[Doug Turner (:dougt)]

This is caused the lack of a notification to the docloader observers by the 
imagelib and/or its customers.  If the docloader observer was notified when a 
request was made (maybe?) by the nsFrameImageLoader, I think that it would fix 
this problem.

Pam, can you get rid of your cache?  We spoke about this briefly:  What impact 
is this in terms of your time and the product performance?

Scott, any ideas about hooking the image layout stuff up to the docloader 
notifications?  

Comment 5

[pnunn]

Doug,
I can't even begin to test out our image cache options
until after beta2. Don't count on it going away. Didn't
the fact that an image has to be approved once before it
gets into the image cache, take care of the problem?

-P

Comment 6

[Doug Turner (:dougt)]

I could, but then an insecure frame in a frameset would be blocked from 
loading.  Special casing images is the wrong thing to do.  The docloader really 
needs to send messages when requesting decompressed images from libimage.  

Comment 7

[Doug Turner (:dougt)]

*** Bug 38675 has been marked as a duplicate of this bug. ***

Comment 8

[Claudius Gayle]

if bug 38675 is a dupe then I should note that this bug also repro's on RH6 Linux as well as WinNT with 2000050909 builds.
Also, the inverse case is also a bug(and easier to repro) i.e. the second visit to a secure site shows a slashed lock.

Comment 9

[Doug Turner (:dougt)]

mscott, images are not build put into the necko loadgroup after they are 
decompressed/loaded.  

Comment 10

[Scott MacGregor]

this sounds like a dup of 33762 to me. 

Comment 11

[Scott MacGregor]

marking as a dup. 

*** This bug has been marked as a duplicate of 33762 ***

Comment 12

[John Unruh]

Verified dupe.

Comment 13

[John Unruh]

Mass changing Security:Crypto to PSM

Comment 14

[John Unruh]

Mass changing Security:Crypto to PSM