Closed
Bug 767916
Opened 12 years ago
Closed 12 years ago
Decode percent encoding on non-ASCII filename when saving a video snapshot.
Categories
(SeaMonkey :: Download & File Handling, defect)
SeaMonkey
Download & File Handling
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.13
People
(Reporter: philip.chee, Assigned: ananuti)
Details
(Whiteboard: [good first bug][mentor=IanN][lang=js][level=oneliner])
Attachments
(1 file)
1.29 KB,
patch
|
iannbugzilla
:
review+
|
Details | Diff | Splinter Review |
From Firefox Bug 734027 (Non-ASCII filename is shown as escaped unicode when saving a snapshot from video): > Steps to reproduce: > 1. Give video file Non-ASCII filename, for example: Basilique du Sacré-Cœur de > Montmartre.ogg > 2. Play this file in Firefox > 3. Right click on the video and select "Save Snapshot as..." > > Actual result: > Snapshot file shows escaped unicode instead of non ASCII characters > > Expected result: > Snapshot file name should contain the correct non-ascii characters. Simple one line fix. See Firefox attachment 604004 [details] [diff] [review]: > - name = url.fileBaseName + ".jpg"; > + name = decodeURI(url.fileBaseName) + ".jpg"; MXR shows the location of the equivalent SeaMonkey code: <http://mxr.mozilla.org/comm-central/search?string=fileBaseName+%2B+".jpg"&find=%2Fsuite%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central>
![]() |
Assignee | |
Comment 1•12 years ago
|
||
Attachment #637371 -
Flags: review?(iann_bugzilla)
![]() |
Assignee | |
Updated•12 years ago
|
Assignee: nobody → ananuti
![]() |
Reporter | |
Updated•12 years ago
|
Status: NEW → ASSIGNED
Comment on attachment 637371 [details] [diff] [review] patch r=me, thanks for the patch
Attachment #637371 -
Flags: review?(iann_bugzilla) → review+
![]() |
Reporter | |
Comment 3•12 years ago
|
||
Hi Ekanan Ketunuti! Do you have checkin access to comm-central or do you need assistance checking this patch in?
Keywords: checkin-needed
Comment 4•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/4c32741c3951
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.13
You need to log in
before you can comment on or make changes to this bug.
Description
•