Closed Bug 375173 Opened 17 years ago Closed 17 years ago

Switch systemLanguage: transient error

Categories

(Core :: SVG, defect)

Product:
Core
Component:
SVG
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: jay, Assigned: longsonr)

References

(
URL
)

Details

(Keywords: testcase)

Attachments

(2 files, 1 obsolete file)

text version
18.12 KB, image/svg+xml
Details
patch
1.28 KB, patch
roc
: review+
roc
: superreview+
Details | Diff | Splinter Review
Attached image text version (obsolete) — 
when not using the default language in switch there is a fairly frequent but transient error where instead of selecting a particular language, a number are displayed.

if necessary change systemLanguage to 'en'
load attachment, note if there is any text that is not 'eggs'
if not, reload page.

the testcase merely duplicates one small code many times.
only three examples of 'eggs' are necessary to show this transient effect regularly.
tested on xp
OS: Mac OS X → All
Hardware: Macintosh → All
Attached image text version — 

    
    

    
  
tighter testcase

it appear that more than a few examples of
  <foreignObject >
  <html xmlns="http://www.w3.org/1999/xhtml" ><body ><object data="" /></body></html>
  </foreignObject>
breaks switch systemLanguage in a transient way.

the data may link without effect to this bug.

    
  

        Attachment #259495 -
        Attachment is obsolete: true

    
    

    
  

      
      
      
      

        Attached patch
          
          
        patch
        — 
          — Splinter Review
      

    


  
I assume this regressed with bug 18333. Before that all the children of a switch would be processed when the switch was processed. Now we can end up processing the extra children of the switch on their own.
Assignee: general → longsonr
Status: NEW → ASSIGNED

        Attachment #271511 -
        Flags: superreview?(roc)

        Attachment #271511 -
        Flags: review?(roc)

    
  
Blocks: incrementalxml

        Attachment #271511 -
        Flags: superreview?(roc)

        Attachment #271511 -
        Flags: superreview+

        Attachment #271511 -
        Flags: review?(roc)

        Attachment #271511 -
        Flags: review+

    
    

    
  
checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED

    
  
Flags: in-testsuite?

    
    

    
  
Looks like the lack of a test here prevented the fact that this patch doesn't actually work (see bug 409383) from being noticed...

    
  
Flags: in-testsuite? → in-testsuite+

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: