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

RESOLVED FIXED in seamonkey2.10

Status

SeaMonkey
Session Restore
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Misak Khachatryan, Assigned: Misak Khachatryan)

Tracking

Trunk
seamonkey2.10
Bug Flags:
in-testsuite +

SeaMonkey Tracking Flags

(seamonkey2.8 fixed, seamonkey2.9 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

11.67 KB, patch
neil@parkwaycc.co.uk
: review+
Details | Diff | Splinter Review
(Assignee)

Description

6 years ago
Created attachment 572306 [details] [diff] [review]
patch

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
(Assignee)

Comment 2

6 years ago
Created attachment 574022 [details] [diff] [review]
fixed patch
Attachment #572306 - Attachment is obsolete: true
Attachment #572306 - Flags: review?(neil)
Attachment #574022 - Flags: review?(neil)

Updated

6 years ago
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

Comment 4

5 years ago
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
(Assignee)

Comment 5

5 years ago
I'm here, just kinda busy :) . Sorry, didn't saw that this is r+ . I'll check this in today.
(Assignee)

Comment 6

5 years ago
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?
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
Flags: in-testsuite+

Comment 7

5 years ago
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?

Updated

5 years ago
Attachment #574022 - Flags: approval-comm-beta?
Attachment #574022 - Flags: approval-comm-beta+
Attachment #574022 - Flags: approval-comm-aurora?
Attachment #574022 - Flags: approval-comm-aurora+

Updated

5 years ago
Keywords: checkin-needed
(Assignee)

Comment 8

5 years ago
http://hg.mozilla.org/releases/comm-aurora/rev/77aa2c8c0468
http://hg.mozilla.org/releases/comm-beta/rev/fcdbfac1e792
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-seamonkey2.8: affected → fixed
status-seamonkey2.9: affected → fixed
Resolution: --- → FIXED
tracking-seamonkey2.8: ? → ---
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.