Removing x-x509-user-cert mime handler.

net/base/filename_util_internal.cc

 // Copyright 2014 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 "net/base/filename_util.h"
 
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/stl_util.h"
 #include "base/strings/string_util.h"
@@ skipping 227 matching lines @@
     HttpContentDisposition header(content_disposition, referrer_charset);
     filename = header.filename();
     if (!filename.empty())
       is_name_from_content_disposition = true;
   }
 
   // Then try to use the suggested name.
   if (filename.empty() && !suggested_name.empty())
     filename = suggested_name;
 
+  if (filename.empty() && mime_type == "application/x-x509-user-cert")
+    filename = "user.crt";

[asanka, 2016/01/11 21:16:34]

Let's move this logic to DownloadTargetDeterminer in //chrome. I.e. let the
DoGenerateTargetPath step in DTD override the filename or the extension based on
the MIME type.

Special casing of application/x-x509-user-cert doesn't belong in //net. The
special handling exists in //chrome.

[svaldez, 2016/01/11 21:34:55]

Done.

+
   // Now try extracting the filename from the URL.  GetFileNameFromURL() only
   // looks at the last component of the URL and doesn't return the hostname as a
   // failover.
   if (filename.empty())
     filename = GetFileNameFromURL(url, referrer_charset, &overwrite_extension);
 
   // Finally try the URL hostname, but only if there's no default specified in
   // |default_name|.  Some schemes (e.g.: file:, about:, data:) do not have a
   // host name.
   if (filename.empty() && default_name.empty() && url.is_valid() &&
@@ skipping 64 matching lines @@
   base::FilePath generated_name(
       base::SysWideToNativeMB(base::UTF16ToWide(file_name)));
 #endif
 
   DCHECK(!generated_name.empty());
 
   return generated_name;
 }
 
 }  // namespace net