Bug 1681924 - add application/zip to extraMimeEntries to help recognize zip files, r=mak

Differential Revision: https://phabricator.services.mozilla.com/D100009

uriloader/exthandler/nsExternalHelperAppService.cpp

@@ -488,6 +488,7 @@ static const nsExtraMimeTypeEntry extraMimeEntries[] = {
     {APPLICATION_GZIP2, "gz", "gzip"},
     {"application/x-arj", "arj", "ARJ file"},
     {"application/rtf", "rtf", "Rich Text Format File"},
+    {APPLICATION_ZIP, "zip", "ZIP Archive"},
     {APPLICATION_XPINSTALL, "xpi", "XPInstall Install"},
     {APPLICATION_PDF, "pdf", "Portable Document Format"},
     {APPLICATION_POSTSCRIPT, "ps,eps,ai", "Postscript File"},

uriloader/exthandler/tests/unit/test_getMIMEInfo_unknown_mime_type.js

@@ -15,18 +15,18 @@ add_task(async function test_check_unknown_mime_type() {
   Assert.equal(zipType, "application/x-zip-compressed");
   try {
     let extension = mimeService.getPrimaryExtension("application/zip", "");
-    Assert.ok(!extension, "Expect no known extension for zip files.");
-  } catch (ex) {
-    Assert.ok(
-      ex,
-      "We'll take an exception instead of an empty string, I guess."
+    Assert.equal(
+      extension,
+      "zip",
+      "Expect our own info to provide an extension for zip files."
     );
+  } catch (ex) {
+    Assert.ok(false, "We shouldn't throw when getting zip info.");
   }
   let found = {};
   let mimeInfo = mimeService.getMIMEInfoFromOS("application/zip", "zip", found);
   Assert.ok(
-    found.value,
-    "Should be able to find mime info even for the unknown mimetype."
+    mimeInfo.hasDefaultHandler,
+    "Should have a default app for zip files"
   );
-  Assert.ok(mimeInfo.hasDefaultHandler, "Should have a default app");
 });