Port Bug 640136 [onchange & input events are not fired for all form elements on restore]

RESOLVED FIXED in seamonkey2.10

Status

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: misak.bugzilla, Assigned: misak.bugzilla)

Tracking

Trunk
seamonkey2.10
Dependency tree / graph
Bug Flags:
in-testsuite +

SeaMonkey Tracking Flags

(seamonkey2.8 fixed, seamonkey2.9 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Posted patch patch (obsolete) — Splinter Review
From parent bug:

Steps:
1. Open the attached testcase
2. Select "2" from the select box (notice a "2" appears below)
3. Restart Firefox
4. Click "Restore Session" from about:home

Result:
2 is selected in select box but the "2" in the div is missing.

Expected:
2 selected in select box and 2 displayed in the div
Attachment #572306 - Flags: review?(neil)
Comment on attachment 572306 [details] [diff] [review]
patch

FYI: There's a slightly modified version of the patch in bug 640136
Posted patch fixed patchSplinter Review
Attachment #572306 - Attachment is obsolete: true
Attachment #572306 - Flags: review?(neil)
Attachment #574022 - Flags: review?(neil)
Attachment #574022 - Flags: review?(neil) → review+
Misak, ping for check-in.
Target Milestone: --- → seamonkey2.10
Serge assuming the patch isn't bit rotted can you check this in for Misak? I don't think he's been around lately.
Keywords: checkin-needed
I'm here, just kinda busy :) . Sorry, didn't saw that this is r+ . I'll check this in today.
Comment on attachment 574022 [details] [diff] [review]
fixed patch

Pushed on trunk, requesting approval for others.

http://hg.mozilla.org/comm-central/rev/2329fac36853
Attachment #574022 - Flags: approval-mozilla-beta?
Attachment #574022 - Flags: approval-mozilla-aurora?
Keywords: checkin-needed
Flags: in-testsuite+
Comment on attachment 574022 [details] [diff] [review]
fixed patch

[Triage Comment]
This landed on comm-central (not m-c), so we can't uplift to Aurora/Beta.
Attachment #574022 - Flags: approval-mozilla-beta?
Attachment #574022 - Flags: approval-mozilla-beta-
Attachment #574022 - Flags: approval-mozilla-aurora?
Attachment #574022 - Flags: approval-mozilla-aurora-
Attachment #574022 - Flags: approval-mozilla-beta-
Attachment #574022 - Flags: approval-mozilla-aurora-
Attachment #574022 - Flags: approval-comm-beta?
Attachment #574022 - Flags: approval-comm-aurora?
Attachment #574022 - Flags: approval-comm-beta?
Attachment #574022 - Flags: approval-comm-beta+
Attachment #574022 - Flags: approval-comm-aurora?
Attachment #574022 - Flags: approval-comm-aurora+
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.