The default bug view has changed. See this FAQ.

Decode percent encoding on non-ASCII filename when saving a video snapshot.

RESOLVED FIXED in seamonkey2.13

Status

SeaMonkey
Download & File Handling
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Philip Chee, Assigned: Ekanan Ketunuti)

Tracking

Trunk
seamonkey2.13
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=IanN][lang=js][level=oneliner])

Attachments

(1 attachment)

(Reporter)

Description

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

5 years ago
Created attachment 637371 [details] [diff] [review]
patch
Attachment #637371 - Flags: review?(iann_bugzilla)
(Assignee)

Updated

5 years ago
Assignee: nobody → ananuti
(Reporter)

Updated

5 years ago
Status: NEW → ASSIGNED

Comment 2

5 years ago
Comment on attachment 637371 [details] [diff] [review]
patch

r=me, thanks for the patch
Attachment #637371 - Flags: review?(iann_bugzilla) → review+
(Reporter)

Comment 3

5 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
https://hg.mozilla.org/comm-central/rev/4c32741c3951
Status: ASSIGNED → RESOLVED
Last Resolved: 5 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.