get rid of unnecessary switch debug console warning

RESOLVED FIXED

Status

Core Graveyard
XForms
--
minor
RESOLVED FIXED
12 years ago
a year ago

People

(Reporter: aaronr, Assigned: aaronr)

Tracking

({fixed1.8.0.5, fixed1.8.1})

Trunk
x86
All
fixed1.8.0.5, fixed1.8.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

1.56 KB, patch
Doron Rosenberg (IBM)
: review+
smaug
: review+
Details | Diff | Splinter Review
1.27 KB, application/xhtml+xml
Details
(Assignee)

Description

12 years ago
I noticed a debug console warning when loading a switch.  It is generated by nsXFormsCaseElement::SetSelected failing due to the fact that when it tries to QI the case element to a nsIXFormsCaseElementUI and it can't because XBL hasn't attached yet.  It is ok that it fails, but the debug console error is annoying me :=)
(Assignee)

Comment 1

12 years ago
Created attachment 217328 [details] [diff] [review]
patch
Attachment #217328 - Flags: superreview?(Olli.Pettay)
Attachment #217328 - Flags: review?(doronr)
(Assignee)

Comment 2

12 years ago
Created attachment 217329 [details]
testcase
(Assignee)

Comment 3

12 years ago
Created attachment 217330 [details]
fixed testcase

first testcase had missing </h2>...ugh!
Attachment #217329 - Attachment is obsolete: true

Updated

12 years ago
Attachment #217328 - Flags: review?(doronr) → review+

Updated

12 years ago
Attachment #217328 - Flags: superreview?(Olli.Pettay) → review+
(Assignee)

Comment 4

12 years ago
checked into trunk on 2006/04/11
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch

Updated

11 years ago
Keywords: fixed1.8.1

Updated

11 years ago
Keywords: fixed1.8.0.5

Updated

11 years ago
Whiteboard: xf-to-branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.