Add an exception to DownloadManager::GenerateExtension() for .tar.gz files wi...

chrome/browser/download/download_manager_unittest.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <string>
 
 #include "chrome/browser/download/download_manager.h"
 #include "chrome/browser/download/download_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 295 matching lines @@
    "http://www.google.com/%EC%98%88%EC%88%A0%20%EC%98%88%EC%88%A0.jpg",
    "image/jpeg",
    L"\uc608\uc220 \uc608\uc220.jpg"},
 
   // Invalid C-D. No filename from URL. Falls back to 'download'.
   {"attachment; filename==?iso88591?Q?caf=E3?",
    "http://www.google.com/path1/path2/",
    "image/jpeg",
    L"download.jpg"},
 
+  // For bug 5772

[sidchat (Google), 2009/03/25 22:09:36]

// Issue = 5772.

+  {"",
+   "http://www.example.com/foo.tar.gz",
+   "application/x-tar",
+   L"foo.tar.gz"},
+
+  {"",
+   "http://www.example.com/bar.tar",
+   "application/x-tar",
+   L"bar.tar"},
+
+  {"",
+   "http://www.example.com/bar.bogus",
+   "application/x-tar",
+   L"bar.bogus.tar"},
+
   // TODO(darin): Add some raw 8-bit Content-Disposition tests.
 };
 
 }  // namespace
 
 // Tests to ensure that the file names we generate from hints from the server
 // (content-disposition, URL name, etc) don't cause security holes.
 TEST_F(DownloadManagerTest, TestDownloadFilename) {
   for (int i = 0; i < arraysize(kGeneratedFiles); ++i) {
     std::wstring file_name;
@@ skipping 51 matching lines @@
 }  // namespace
 
 TEST_F(DownloadManagerTest, GetSafeFilename) {
   for (int i = 0; i < arraysize(kSafeFilenameCases); ++i) {
     FilePath path(kSafeFilenameCases[i].path);
     download_manager_->GenerateSafeFilename(kSafeFilenameCases[i].mime_type,
         &path);
     EXPECT_EQ(kSafeFilenameCases[i].expected_path, path.value());
   }
 }