select1 initial size wrong

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
2 years ago

People

(Reporter: doronr, Assigned: smaug)

Tracking

({fixed1.8.0.2, fixed1.8.1})

Trunk
x86
All
fixed1.8.0.2, fixed1.8.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

13 years ago
var popupBox = document.getBoxObjectFor(this.popup);
var w = popupBox.width;

returns 0 and 2 (2 calls) on startup with this testcase
(Reporter)

Comment 1

13 years ago
Created attachment 200906 [details]
testcase
(Reporter)

Comment 2

13 years ago
In this testcase (linux and windows, branch and trunk), the select1 is not wide enough.  Changing the instance data using the input fixes the issue.
(Assignee)

Updated

13 years ago
Status: NEW → ASSIGNED
Created attachment 201048 [details] [diff] [review]
Remove wrong optimization

The "optimization" in 
http://lxr.mozilla.org/seamonkey/source/extensions/xforms/nsXFormsDelegateStub.cpp#106
should be enough.
Attachment #201048 - Flags: review?(doronr)
(Assignee)

Updated

13 years ago
Attachment #201048 - Flags: superreview?
Attachment #201048 - Flags: review?(allan)
(Reporter)

Updated

13 years ago
Attachment #201048 - Flags: superreview?
Attachment #201048 - Flags: review?(doronr)
Attachment #201048 - Flags: review+

Updated

13 years ago
Attachment #201048 - Flags: review?(allan) → review+
Checked in
Whiteboard: xf-to-branch

Comment 5

13 years ago
checked into MOZILLA_1_8_BRANCH via bug 323691.  Leaving open for now until it gets into 1.8.0

Updated

13 years ago
Whiteboard: xf-to-branch

Updated

13 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Keywords: fixed1.8.0.2
Resolution: --- → FIXED

Comment 6

13 years ago
verfied fixed on MOZILLA_1_8_BRANCH
Keywords: fixed1.8.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.