Pass content-type resources to web intents. Goes through download, then invokes the p…

content/browser/renderer_host/buffered_resource_handler.cc

Index: content/browser/renderer_host/buffered_resource_handler.cc
diff --git a/content/browser/renderer_host/buffered_resource_handler.cc b/content/browser/renderer_host/buffered_resource_handler.cc
index 7feb15421a8356d25824cc336f24b0a3066f565f..0bd124c370de0014c71cde7cbe3cdaeeb92011ef 100644
--- a/content/browser/renderer_host/buffered_resource_handler.cc
+++ b/content/browser/renderer_host/buffered_resource_handler.cc
@@ -183,6 +183,7 @@ bool BufferedResourceHandler::DelayResponse() {
     response_->mime_type.assign(mime_type);
   }
 
+  /*

[darin (slow to review), 2012/03/10 01:02:14]

are these comments intended?

[Greg Billock, 2012/03/13 17:56:10]

They're needed to enable feed content type handling. Currently we're "handling"
these types by overwriting the type to plain text.

   if (mime_type == "application/rss+xml" ||
       mime_type == "application/atom+xml") {
     // Sad face.  The server told us that they wanted us to treat the response
@@ -195,6 +196,7 @@ bool BufferedResourceHandler::DelayResponse() {
     mime_type.assign("text/plain");
     response_->mime_type.assign(mime_type);
   }
+  */
 
   if (!not_modified_status && ShouldWaitForPlugins()) {
     wait_for_plugins_ = true;
@@ -360,6 +362,12 @@ bool BufferedResourceHandler::ShouldDownload(bool* need_plugin_list) {
       host_->delegate()->ShouldForceDownloadResource(request_->url(), type))
     return true;
 
+  // mime type support currently says it supports feed types.
+  if (type == "application/rss+xml" ||
+      type == "application/atom+xml") {
+    return true;
+  }
+
   // MIME type checking.
   if (net::IsSupportedMimeType(type))
     return false;