Closed Bug 332872 Opened 18 years ago Closed 18 years ago

get rid of unnecessary switch debug console warning

Categories

(Core Graveyard :: XForms, defect)

x86
All
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aaronr, Assigned: aaronr)

Details

(Keywords: fixed1.8.0.5, fixed1.8.1)

Attachments

(2 files, 1 obsolete file)

1.56 KB, patch
doronr
: review+
smaug
: review+
Details | Diff | Splinter Review
1.27 KB, application/xhtml+xml
Details
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 :=)
Attached patch patchSplinter Review
Attachment #217328 - Flags: superreview?(Olli.Pettay)
Attachment #217328 - Flags: review?(doronr)
Attached file testcase (obsolete) —
Attached file fixed testcase
first testcase had missing </h2>...ugh!
Attachment #217329 - Attachment is obsolete: true
Attachment #217328 - Flags: review?(doronr) → review+
Attachment #217328 - Flags: superreview?(Olli.Pettay) → review+
checked into trunk on 2006/04/11
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
Keywords: fixed1.8.1
Keywords: fixed1.8.0.5
Whiteboard: xf-to-branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: