Switch systemLanguage: transient error

RESOLVED FIXED

Status

()

RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: jay, Assigned: longsonr)

Tracking

({testcase})

Trunk
testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

12 years ago
Created attachment 259495 [details]
text version

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.
(Reporter)

Comment 1

12 years ago
tested on xp
OS: Mac OS X → All
Hardware: Macintosh → All
(Reporter)

Comment 2

12 years ago
Created attachment 259497 [details]
text version
(Reporter)

Comment 3

12 years ago
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.
(Reporter)

Updated

12 years ago
Attachment #259495 - Attachment is obsolete: true
(Assignee)

Comment 4

12 years ago
Created attachment 271511 [details] [diff] [review]
patch

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)
(Assignee)

Updated

12 years ago
Blocks: 18333
Attachment #271511 - Flags: superreview?(roc)
Attachment #271511 - Flags: superreview+
Attachment #271511 - Flags: review?(roc)
Attachment #271511 - Flags: review+
(Assignee)

Comment 5

12 years ago
checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Updated

12 years ago
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...
(Assignee)

Updated

11 years ago
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.