48515 - incorrectly getting OnStateChange notifications for chrome

Status: NEW

(Core :: DOM: Navigation, defect, P3)

Description

[John G. Myers]

setenv NSPR_LOG_MODULES nsSecureBroswerUI:5

(note misspelling), make the window small and visit 
https://scopus.mcom.com/bugsplat  The window has to be small enough so that 
loading the page will cause both the horizontal and vertical scrollbars to 
appear.

Note that the nsSecureBrowserUIImpl::OnStateChange() observer is being notified 
of chrome loads for the scrollbars.  [e-mail] says that this is a 
regression.

Comment 1

[John G. Myers]

The spelling is now fixed:

setenv NSPR_LOG_MODULES nsSecureBrowserUI:5

Comment 2

[selmer (gone)]

So should this be closed fixed?

Comment 3

[John G. Myers]

No, the bug is still there.

Comment 4

[Doug Turner (:dougt)]

we rick potts promotes the logic in psmglue to the docloader this problem will 
go away.  cc-ing him.

Comment 5

[John G. Myers]

Moving to Embedding: Docshell

Comment 6

[Judson Valeski]

John,
This is a result of psm registering via the nsIDocLoader service, can it access
a nsIDOMWindow at some point and register w/ that window instead?

Comment 7

[Judson Valeski]

moving out. although this isn't optimal, it shouldn't be hurting anything.

Comment 8

[Asa Dotzler [:asa]]

Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)

Comment 9

[John Unruh]

Any progress on this 2 year old bug?

Comment 10

[Vedran Miletic]

retargeting

Comment 11

[benc]

->defaults.