Closed Bug 332872 Opened 17 years ago Closed 17 years ago

get rid of unnecessary switch debug console warning

Categories

(Core Graveyard :: XForms, defect)

Product:
Core Graveyard
Component:
XForms
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: aaronr, Assigned: aaronr)

Details

(Keywords: fixed1.8.0.5, fixed1.8.1)

Attachments

(2 files, 1 obsolete file)

patch
1.56 KB, patch
doronr
: review+
smaug
: review+
Details | Diff | Splinter Review
fixed testcase
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 patch — — Splinter 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: 17 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.