Ignore non-ASCII headers in the mimeHandlerPrivate API.

extensions/browser/api/mime_handler_private/mime_handler_private.cc

Index: extensions/browser/api/mime_handler_private/mime_handler_private.cc
diff --git a/extensions/browser/api/mime_handler_private/mime_handler_private.cc b/extensions/browser/api/mime_handler_private/mime_handler_private.cc
index b7d557d18015285cb3bb90a542fb863dfdf0bfaa..e9d3cefa46378c8e4e8235252aca0c510172872c 100644
--- a/extensions/browser/api/mime_handler_private/mime_handler_private.cc
+++ b/extensions/browser/api/mime_handler_private/mime_handler_private.cc
@@ -4,6 +4,7 @@
 
 #include "extensions/browser/api/mime_handler_private/mime_handler_private.h"
 
+#include "base/strings/string_util.h"
 #include "content/public/browser/stream_handle.h"
 #include "content/public/browser/stream_info.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.h"
@@ -24,6 +25,10 @@ mojo::Map<mojo::String, mojo::String> CreateResponseHeadersMap(
   std::string header_name;
   std::string header_value;
   while (headers->EnumerateHeaderLines(&iter, &header_name, &header_value)) {
+    // mojo strings must be UTF-8 and headers might not be, so drop any headers
+    // that aren't ASCII.

[raymes, 2015/04/20 23:18:45]

I think we should add a comment with justification for why we can drop these
headers.

[Sam McNally, 2015/04/21 00:45:32]

Done.

+    if (!base::IsStringASCII(header_name) || !base::IsStringASCII(header_value))
+      continue;
     auto& current_value = result[header_name];
     if (!current_value.empty())
       current_value += ", ";