Closed Bug 376931 Opened 17 years ago Closed 17 years ago

output mediatype patch won't work on branches

Categories

(Core Graveyard :: XForms, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aaronr, Assigned: aaronr)

References

()

Details

(Keywords: fixed1.8.0.12, fixed1.8.1.4)

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
The current code we have to implement output mediatype in the trunk won't work on the branches.  Per comment in bug 376817, we probably shouldn't change the anonymous content's DOM while the XBL is attaching which is what we are currently doing for output with mediatype.

Testcase to verify the fix is listed in URL and here: https://bugzilla.mozilla.org/attachment.cgi?id=232362

This patch uses a setTimeout call to set the value of the underlying anonymous content.
Attachment #261039 - Flags: review?(surkov.alexander)
Attachment #261039 - Flags: review?(doronr)
Comment on attachment 261039 [details] [diff] [review]
patch

>+              // FF 1.5 or FF 2.  Wo we'll wait a tick before changing it here.

s/Wo/So when you check in please :)
Attachment #261039 - Flags: review?(doronr) → review+
Attached patch patch2Splinter Review
fixes Doron's nit
Attachment #261039 - Attachment is obsolete: true
Attachment #261054 - Flags: review?(surkov.alexander)
Attachment #261039 - Flags: review?(surkov.alexander)
Attachment #261054 - Flags: review?(surkov.alexander) → review+
checked into trunk
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
checked into 1.8 branch on 2007-04-12
checked into 1.8.0 branch on 2007-04-16
Whiteboard: xf-to-branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: