Closed Bug 700144 Opened 13 years ago Closed 13 years ago

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

Categories

(SeaMonkey :: Session Restore, defect)

Product:
SeaMonkey
Component:
Session Restore
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(seamonkey2.8 fixed, seamonkey2.9 fixed)

Status:
RESOLVED FIXED
Milestone:
seamonkey2.10
Tracking Flags:
Tracking Status
seamonkey2.8 --- fixed
seamonkey2.9 --- fixed

People

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

References

Details

Attachments

(1 file, 1 obsolete file)

fixed patch
11.67 KB, patch
neil
: review+
iannbugzilla
: approval-comm-aurora+
iannbugzilla
: approval-comm-beta+
Details | Diff | Splinter Review
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 patch — — Splinter 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.
status-seamonkey2.8: --- → affected
status-seamonkey2.9: --- → affected
tracking-seamonkey2.8: --- → ?
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.

Attachment

General

Creator:
Created:
Updated:
Size: