Hidden views that are later shown don't have transparency checked

VERIFIED FIXED

Status

()

Core
CSS Parsing and Computation
P3
normal
VERIFIED FIXED
19 years ago
a year ago

People

(Reporter: David Hyatt, Assigned: troy)

Tracking

Trunk
x86
Other
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: 2/21: 4th request for verification from assigned engineer or reporteer)

(Reporter)

Description

19 years ago
Hidden views are marked as having transparent content.  When a view is later
shown as a result of a style change, the view's visibility is updated, but its
transparency isn't rechecked.

CreateViewForFrame in nsHTMlContainerFrame is where the initial setting of
transparency to TRUE happens for hidden views.  The SetVisibility method over in
the view code is where the view gets shown, and no check is happening there to
possibly clear the transparency flag.

Updated

19 years ago
Assignee: peterl → troy

Comment 1

19 years ago
Need to add transparency code to ApplyRenderingChangeToTree as well as DidReflow
(Assignee)

Comment 2

19 years ago
There's also no test case associated with this bug...
(Assignee)

Updated

19 years ago
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

19 years ago
Added code to nsFrame::DidReflow() and SyncAndInvalidateView() to do the same
thing as CreateViewForFrame()

Updated

19 years ago
Whiteboard: 9/30: Requested verification from assigned engineer or reporteer

Comment 4

19 years ago
hyatt@netscape.com or troy@netscape.com: Please mark bug VERIFIED. There is
insufficient information for me to do so. Thanks

Updated

18 years ago
Whiteboard: 9/30: Requested verification from assigned engineer or reporteer → 11/16: 2nd request: Requested verification from assigned engineer or reporteer

Comment 5

18 years ago
2nd request: Requesting verification from Reporter or Assigned engineer -
insufficient info for me to do so.

Updated

18 years ago
Whiteboard: 11/16: 2nd request: Requested verification from assigned engineer or reporteer → 12/3: Requested verification from assigned engineer or reporteer

Comment 6

18 years ago
Troy or David: Please verify this bug fixed - I am unable to do so. Thanks

Comment 7

18 years ago
Request verification by assigned engineer or reporter - please!
Whiteboard: 12/3: Requested verification from assigned engineer or reporteer → 2/21: 4th request for verification from assigned engineer or reporteer

Comment 8

18 years ago
After 4 requests for verification and no response, I am marking this verified. 
If it is still a problem, please reopen.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.