Closed Bug 282265 Opened 19 years ago Closed 19 years ago

tree.view not directly available after it is set hidden=false, when it was first hidden=true

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.8beta1

People

(Reporter: martijn.martijn, Assigned: bzbarsky)

References

Details

(Keywords: testcase)

Attachments

(2 files)

This issue caused bug 260951.
I'm not sure if it is a bug, feel free to resolve it INVALID, if you think so.

I'll attach a testcase.
When you click for the first time on the "alert(tree.view)" button, it shows 'null'.
But in older builds (before 2004-08-10) it showed '[object XULTreeBuilder]'
already for the first time.

This worked in Firefox build 2004-08-09 06:48am.
But it doesn't work anymore in Firefox build 2004-08-10 06:49am.
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2004-08-09+06%3A48%3A00&maxdate=2004-08-10+06%3A49%3A00&cvsroot=%2Fcvsroot

I think this has changed because of the fix of bug 230170.
Attached file Testcase
Hmm.. the patch for bug 260951 looks like a nasty hack that could use backing
out... ;)

I think the right fix is to fix nsBoxObject::GetFrame to flush out frame
construction when it's called.  Now the question is, how often is it called from
inside frame code?
Attached patch Patch, should beSplinter Review
Attachment #174334 - Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #174334 - Flags: review?(mconnor)
Attachment #174334 - Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Comment on attachment 174334 [details] [diff] [review]
Patch, should be

*cough* this was my master plan all along.... 

Thanks. :)
Attachment #174334 - Flags: review?(mconnor) → review+
Comment on attachment 174334 [details] [diff] [review]
Patch, should be

Could this be approved for 1.8b?  This is fallout from back in the summer, and
affects all box object users...  The patch should be pretty safe -- I've
checked our frame code to make sure this function is not called from inside
frame implementations...
Attachment #174334 - Flags: approval1.8b?
Comment on attachment 174334 [details] [diff] [review]
Patch, should be

a=asa for 1.8b
Attachment #174334 - Flags: approval1.8b? → approval1.8b+
Assignee: nobody → bzbarsky
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → mozilla1.8beta1
Fixed for 1.8b1.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Unfortunately, this does not appear to fix the original issue from bug 260951.
Ah, indeed.  The flush is ignored, since we're inside frame construction...  :(

The change in bug 282359 will help with that; if that doesn't land soon, I'll
undo the JS changes in this bug.
Depends on: 282359
Blocks: 230170
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: