Closed Bug 700144 Opened 9 years ago Closed 8 years ago

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

Categories

(SeaMonkey :: Session Restore, defect)

defect
Not set
normal

Tracking

(seamonkey2.8 fixed, seamonkey2.9 fixed)

RESOLVED FIXED
seamonkey2.10
Tracking Status
seamonkey2.8 --- fixed
seamonkey2.9 --- fixed

People

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

References

Details

Attachments

(1 file, 1 obsolete file)

Attached 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
Attached 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.