Removing x-x509-user-cert mime handler.

chrome/browser/download/chrome_download_manager_delegate.cc

Index: chrome/browser/download/chrome_download_manager_delegate.cc
diff --git a/chrome/browser/download/chrome_download_manager_delegate.cc b/chrome/browser/download/chrome_download_manager_delegate.cc
index 3d609fb2e733224ca63ce2239f094415f6a16373..4e4a18cb7d7c7d9c59d205280b01663341f0336a 100644
--- a/chrome/browser/download/chrome_download_manager_delegate.cc
+++ b/chrome/browser/download/chrome_download_manager_delegate.cc
@@ -465,6 +465,13 @@ void ChromeDownloadManagerDelegate::OpenDownload(DownloadItem* download) {
       NEW_FOREGROUND_TAB,
       ui::PAGE_TRANSITION_LINK,
       false);
+
+  if (download->GetOriginalMimeType() == "application/x-x509-user-cert") {
+    params = content::OpenURLParams(GURL("chrome://settings/certificates"),

[Ryan Sleevi, 2015/12/08 02:38:37]

There's no way to smuggle the file name or pre-warm the UI, right?

[I would imagine not, but just double checking; we may be able to hack something
with URL fragments or query strings, but that's probably not necessary yet]

[svaldez, 2015/12/08 15:43:39]

We could pipe this through, though we'll end up having the full filename in the
URL fragment. If that's reasonable, I can create a CL for that and we can see
whether we want to use it.

[svaldez, 2015/12/08 16:34:35]

So attempting to pipe this through, there isn't really a good way of having the
UI default to a specific file. We can trigger the importPersonal file selector
with the correct file, but then the UX is that a Open File dialog popups and the
user really isn't informed of what the file dialog is for. If we bypass file
selection entirely, then there really isn't a chance for the user to
cancel/accept the import.

[Ryan Sleevi, 2015/12/09 01:27:24]

Fair point. Short of reworking this UI, which we could do later, I think it's
fine as is.

+                                    content::Referrer(), NEW_FOREGROUND_TAB,
+                                    ui::PAGE_TRANSITION_LINK, false);
+  }
+
   browser->OpenURL(params);
   RecordDownloadOpenMethod(DOWNLOAD_OPEN_METHOD_DEFAULT_BROWSER);
 #else
@@ -720,6 +727,11 @@ void ChromeDownloadManagerDelegate::OnDownloadTargetDetermined(
         target_info->is_filetype_handled_safely)
       DownloadItemModel(item).SetShouldPreferOpeningInBrowser(true);
 
+#if defined(OS_LINUX) || defined(OS_CHROMEOS)
+    if (item->GetOriginalMimeType() == "application/x-x509-user-cert")
+      DownloadItemModel(item).SetShouldPreferOpeningInBrowser(true);
+#endif
+
     if (target_info->is_dangerous_file)
       DownloadItemModel(item).SetIsDangerousFileBasedOnType(true);
   }