Removing x-x509-user-cert mime handler.

content/browser/loader/mime_type_resource_handler.cc

 // Copyright (c) 2012 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 "content/browser/loader/mime_type_resource_handler.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
 #include "base/thread_task_runner_handle.h"
 #include "components/mime_util/mime_util.h"
 #include "content/browser/download/download_resource_handler.h"
 #include "content/browser/download/download_stats.h"
 #include "content/browser/loader/certificate_resource_handler.h"
 #include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/browser/loader/resource_request_info_impl.h"
 #include "content/browser/loader/stream_resource_handler.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/download_item.h"
@@ skipping 349 matching lines @@
     if (mime_util::IsSupportedMimeType(mime_type))
       return true;
 
     bool handled_by_plugin;
     if (!SelectPluginHandler(defer, &handled_by_plugin))
       return false;
     if (handled_by_plugin || *defer)
       return true;
   }
 
+  scoped_ptr<DownloadSaveInfo> save_info(new DownloadSaveInfo);
+  if (base::ToLowerASCII(mime_type) == "application/x-x509-user-cert")
+    save_info->suggested_name = base::UTF8ToUTF16("user_cert.crt");

[mmenke, 2015/10/29 15:34:56]

Why not put this in GetSuggestedFilename?  Also, why are you setting more than
just the extension?

[svaldez, 2015/10/29 16:32:19]

Done.

+
   // Install download handler
   info->set_is_download(true);
   scoped_ptr<ResourceHandler> handler(
       host_->CreateResourceHandlerForDownload(
           request(),
           true,  // is_content_initiated
           must_download,
           DownloadItem::kInvalidId,
-          scoped_ptr<DownloadSaveInfo>(new DownloadSaveInfo()),
+          save_info.Pass(),
           DownloadUrlParameters::OnStartedCallback()));
   return UseAlternateNextHandler(handler.Pass(), std::string());
 }
 
 bool MimeTypeResourceHandler::UseAlternateNextHandler(
     scoped_ptr<ResourceHandler> new_handler,
     const std::string& payload_for_old_handler) {
   if (response_->head.headers.get() &&  // Can be NULL if FTP.
       response_->head.headers->response_code() / 100 != 2) {
     // The response code indicates that this is an error page, but we don't
@@ skipping 111 matching lines @@
   request()->LogUnblocked();
   bool defer = false;
   if (!ProcessResponse(&defer)) {
     controller()->Cancel();
   } else if (!defer) {
     controller()->Resume();
   }
 }
 
 }  // namespace content