Closed
Bug 375173
Opened 17 years ago
Closed 17 years ago
Switch systemLanguage: transient error
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: jay, Assigned: longsonr)
References
()
Details
(Keywords: testcase)
Attachments
(2 files, 1 obsolete file)
18.12 KB,
image/svg+xml
|
Details | |
1.28 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
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 2•17 years ago
|
||
Reporter | ||
Comment 3•17 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•17 years ago
|
Attachment #259495 -
Attachment is obsolete: true
Assignee | ||
Comment 4•17 years ago
|
||
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•17 years ago
|
Blocks: incrementalxml
Attachment #271511 -
Flags: superreview?(roc)
Attachment #271511 -
Flags: superreview+
Attachment #271511 -
Flags: review?(roc)
Attachment #271511 -
Flags: review+
Assignee | ||
Comment 5•17 years ago
|
||
checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Flags: in-testsuite?
Comment 6•17 years ago
|
||
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•16 years ago
|
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•