select1 initial size wrong

RESOLVED FIXED

Status

Core Graveyard
XForms
RESOLVED FIXED
12 years ago
a year ago

People

(Reporter: Doron Rosenberg (IBM), 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)

2.39 KB, application/xhtml+xml
Details
1.20 KB, patch
Doron Rosenberg (IBM)
: review+
Allan Beaufour
: review+
Details | Diff | Splinter Review
(Reporter)

Description

12 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

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

Comment 2

12 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

12 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 3

12 years ago
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

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

Updated

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

Updated

12 years ago
Attachment #201048 - Flags: review?(allan) → review+
(Assignee)

Comment 4

12 years ago
Checked in
Whiteboard: xf-to-branch

Comment 5

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

Updated

12 years ago
Whiteboard: xf-to-branch

Updated

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

Comment 6

11 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.