[Issue - 116972] - Save File Problem with tar.bz

Downloading a file with ".tar.bz" extension mulitple times resulted in incorrect filename where ordinal number was suffixed to "tar" instead of filename as below.

Actual - file.tar(1).bz

Expected - file(1).tar.bz

This happened because "bz" was not recognized as double extension.

BUG=116972
Committed: https://crrev.com/fad66c3012d0c95b447f5bc46da7602f57188dfe
Cr-Commit-Position: refs/heads/master@{#298789}

[arun87.kumar, 2014-10-07 03:29:15]

arun87.kumar@samsung.com changed reviewers:
+ asanka@chromium.org, rdsmith@chromium.org, rvargas@chromium.org

[arun87.kumar, 2014-10-07 03:32:31]

Hi all,

I have made changes to fix the above issue as per asanka's comments.

I have locally tested the patch and it is working fine.


PTAL







Regards,
Arun

[asanka, 2014-10-07 03:52:35]

code lgtm

Please revise the commit message as per
http://dev.chromium.org/developers/contributing-code#TOC-Change-List-Descript...

How are you uploading your patches? The 'git cl upload' command will prepare a
commit message that has at least a BUG= line on first upload. Running 'git cl
description' after an upload launches an editor with some directions in comments
about how to format a commit message.

[asanka, 2014-10-07 03:56:13]

On 2014/10/07 03:52:35, asanka wrote:
> code lgtm
> 
> Please revise the commit message as per
>
http://dev.chromium.org/developers/contributing-code#TOC-Change-List-Descript...
> 
> How are you uploading your patches? The 'git cl upload' command will prepare a
> commit message that has at least a BUG= line on first upload. Running 'git cl
> description' after an upload launches an editor with some directions in
comments
> about how to format a commit message.

You'll also need OWNERS approval for this change.

[arun87.kumar, 2014-10-07 04:37:35]

On 2014/10/07 03:56:13, asanka wrote:
> On 2014/10/07 03:52:35, asanka wrote:
> > code lgtm
> > 
> > Please revise the commit message as per
> >
>
http://dev.chromium.org/developers/contributing-code#TOC-Change-List-Descript...
> > 
> > How are you uploading your patches? The 'git cl upload' command will prepare
a
> > commit message that has at least a BUG= line on first upload. Running 'git
cl
> > description' after an upload launches an editor with some directions in
> comments
> > about how to format a commit message.
> 
> You'll also need OWNERS approval for this change.

asanka@

Thanks for your quick review. I have updated the CL Description with BUG id.
For owners approval, i have added rvargas as reviewer.



rvargas@

PTAL.



Regards,
Arun

[rvargas (doing something else), 2014-10-08 00:38:13]

lgtm

[rvargas (doing something else), 2014-10-08 00:38:19]

The CQ bit was checked by rvargas@chromium.org

[commit-bot: I haz the power, 2014-10-08 00:39:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/630323003/1

[rvargas (doing something else), 2014-10-08 00:41:01]

The CQ bit was unchecked by rvargas@chromium.org

[rvargas (doing something else), 2014-10-08 00:50:17]

Arun,

You are not yet listed as an authorized contributor in the current Samsung CLA.

(http://www.chromium.org/developers/contributing-code/external-contributor-che...)

[arun87.kumar, 2014-10-08 06:26:52]

On 2014/10/08 00:50:17, rvargas wrote:
> Arun,
> 
> You are not yet listed as an authorized contributor in the current Samsung
CLA.
> 
>
(http://www.chromium.org/developers/contributing-code/external-contributor-che...)

rvargas@

Thanks for your quick review. I have added my id in AUTHORS file. AFAIK, i had
requested Laszlo Gombos for CLA. 


laszlo@

Please can you confirm if i am there on Samsung CLA?



Regards,
Arun

[asanka, 2014-10-08 15:49:19]

On 2014/10/08 00:50:17, rvargas wrote:
> Arun,
> 
> You are not yet listed as an authorized contributor in the current Samsung
CLA.
> 
>
(http://www.chromium.org/developers/contributing-code/external-contributor-che...)

Whoa. Thanks for checking. I was under the impression that Samsung had a
corporate CLA.

[lgombos, 2014-10-08 18:06:24]

On 2014/10/08 00:50:17, rvargas wrote:

> You are not yet listed as an authorized contributor in the current Samsung
CLA.
>
(http://www.chromium.org/developers/contributing-code/external-contributor-che...)

Can you double-check ? Max confirmed that Arun is on the CLA on September 03,
2014.

[rvargas (doing something else), 2014-10-08 18:51:20]

On 2014/10/08 18:06:58, lgombos wrote:

Interesting. There are two lists... one for "Samsung" and another one for
"Samsung Electronics" (and they are not consecutive on the document). Sure
enough, I checked the former yesterday but Arun is in the latter.

[rvargas (doing something else), 2014-10-08 18:51:38]

The CQ bit was checked by rvargas@chromium.org

[commit-bot: I haz the power, 2014-10-08 18:54:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/630323003/20001

[commit-bot: I haz the power, 2014-10-09 04:50:52]

Committed patchset #2 (id:20001) as 779b5fa66f86ead21c0718e6422981f65a5ef1f4

[commit-bot: I haz the power, 2014-10-09 04:52:12]

Patchset 2 (id:??) landed as
https://crrev.com/fad66c3012d0c95b447f5bc46da7602f57188dfe
Cr-Commit-Position: refs/heads/master@{#298789}