Browse Source

Fix a missing the specified mime type check in <input> accept attribute.

Haojian Wu 9 years ago
parent
commit
617bff8ec8
1 changed files with 6 additions and 5 deletions
  1. 6 5
      atom/browser/web_dialog_helper.cc

+ 6 - 5
atom/browser/web_dialog_helper.cc

@@ -43,14 +43,15 @@ file_dialog::Filters GetFileTypesFromAcceptType(
       // Skip the first character.
       extensions.push_back(extension.substr(1));
     } else {
-      if (ascii_type == "image/*" || ascii_type == "audio/*" ||
-          ascii_type == "video/*") {
-        // For MIME Type
-        net::GetExtensionsForMimeType(ascii_type, &extensions);
-      }
+      // For MIME Type, `audio/*, vidio/*, image/*
+      net::GetExtensionsForMimeType(ascii_type, &extensions);
     }
   }
 
+  // If no valid exntesion is added, return empty filters.
+  if (extensions.empty())
+    return filters;
+
   filters.push_back(file_dialog::Filter());
   for (const auto& extension : extensions) {
 #if defined(OS_WIN)