Closed Bug 375173 Opened 15 years ago Closed 15 years ago

Switch systemLanguage: transient error

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jay, Assigned: longsonr)

References

()

Details

(Keywords: testcase)

Attachments

(2 files, 1 obsolete file)

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 patchSplinter 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)
Attachment #271511 - Flags: superreview?(roc)
Attachment #271511 - Flags: superreview+
Attachment #271511 - Flags: review?(roc)
Attachment #271511 - Flags: review+
checked in.
Status: ASSIGNED → RESOLVED
Closed: 15 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.