Closed Bug 547710 Opened 10 years ago Closed 10 years ago

Port Bug 523771 [Support <input type=file multiple>] sessionstore part to SeaMonkey

Categories

(SeaMonkey :: Session Restore, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a1

People

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

References

Details

Attachments

(2 files)

From parent bug:

"If the 'multiple' attribute is set on an <input type=file> then we should use a
filepicker that allows multiple files to be selected.

All the files should then be used when submitting, and be available through the
HTMLInputElement.files properly."

This is only sessionstore part, tests for this bug will be in coming sessionstore tests patch.
Attachment #428192 - Flags: superreview?(neil)
Attachment #428192 - Flags: review?(neil)
Attachment #428192 - Attachment is patch: true
Attachment #428192 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 428192 [details] [diff] [review]
patch
[Checkin: Comment 3]

I wonder why nobody complained about backward compatibility, but I guess that keeping file input data across versions is a low priority ;-)
Attachment #428192 - Flags: superreview?(neil)
Attachment #428192 - Flags: superreview+
Attachment #428192 - Flags: review?(neil)
Attachment #428192 - Flags: review+
Keywords: checkin-needed
Neil, I just found this -  Bug 524533. Is it worth new bug or I can post modified patch for checkin here ?
Comment on attachment 428192 [details] [diff] [review]
patch
[Checkin: Comment 3]


http://hg.mozilla.org/comm-central/rev/70684cdfcd65
Attachment #428192 - Attachment description: patch → patch [Checkin: Comment 3]
(In reply to comment #2)
> Bug 524533. Is it worth new bug ... ?

Here is fine.
Depends on: 524533
Flags: in-testsuite?
Keywords: checkin-needed
Target Milestone: --- → seamonkey2.1a1
One line patch.
Attachment #428427 - Flags: superreview?(neil)
Attachment #428427 - Flags: review?(neil)
Attachment #428427 - Flags: superreview?(neil)
Attachment #428427 - Flags: superreview+
Attachment #428427 - Flags: review?(neil)
Attachment #428427 - Flags: review+
Keywords: checkin-needed
Comment on attachment 428427 [details] [diff] [review]
mozGetFileNameArray length out param should be optional
[Checkin: Comment 6]


http://hg.mozilla.org/comm-central/rev/28d0237c1df8
Attachment #428427 - Attachment description: mozGetFileNameArray length out param should be optional → mozGetFileNameArray length out param should be optional [Checkin: Comment 6]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.