304338 - nsWindowSH doesn't allow xpcom components to replace its properties

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement)

Description

[timeless]

The code tries and fails to set security flags so that things work, 
unfortunately the code isn't reentrant safe and reenters, resulting in 
clobbered values.

Comment 1

[timeless]

Attachment: make the code reentrancy safe

Comment 2

[Johnny Stenback (:jst)]

Comment on attachment 192398 [details] [diff] [review]
make the code reentrancy safe

r+sr=jst

Comment 3

[timeless]

Comment on attachment 192398 [details] [diff] [review]
make the code reentrancy safe

mozilla/dom/src/base/nsDOMClassInfo.cpp 	1.295

this change is safe, small, tested, has been on trunk for a while, and we need
it for the branch

Comment 4

[Chris Beard]

Comment on attachment 192398 [details] [diff] [review]
make the code reentrancy safe

-'d approval1.8b4 request.  to get onto the branch at this point bugs needs to
be nominated for 1.8b4.  also, it's unclear in this bug what this will actually
fix and why we would need this.

Comment 5

[Brendan Eich [:brendan]]

Comment on attachment 192398 [details] [diff] [review]
make the code reentrancy safe

Timeless, request that the bug be marked blocking1.8b4 first, get drivers to
agree (I'll agree, but at this point I will want a second).

This is a safe fix that I'm sure at least one company building on the platform
wants; its bug ought to be considered for 1.8, even at this point.  Don't let
the "enhancement" status fool you, fellow drivers.

/be

Comment 6

[Brendan Eich [:brendan]]

See comment 5.

/be

Comment 7

[timeless]

sorry, i'm 4000 bugmails behind, i just got to this one. my company needs this
change, it's blocking our product which is being built on 1.8.

Comment 8

[Brendan Eich [:brendan]]

Comment on attachment 192398 [details] [diff] [review]
make the code reentrancy safe

Urging reconsideration, for b4 or b5 -- this is a safe fix that makes JS API
usage work the way it should.

/be

Comment 9

[Asa Dotzler [:asa]]

Time is short for 1.8b4. If this isn't landed today, it's not going to make the
train.

Comment 10

[timeless]

mozilla/dom/src/base/nsDOMClassInfo.cpp 	1.292.2.8