Bug 1160200 - APNG can't be used with type switching. r=mcaceres, sr=jrmuizel

nwtn · nwtn · commit 5c07d1ccdbc9 · 2015-09-25T11:36:00.000+02:00
--HG--
extra : rebase_source : ea7ff84295608a51d663fee48707bfec5a8c13e5
diff --git a/image/DecoderFactory.cpp b/image/DecoderFactory.cpp
@@ -34,6 +34,8 @@ DecoderFactory::GetDecoderType(const char* aMimeType)
     type = DecoderType::PNG;
   } else if (!strcmp(aMimeType, IMAGE_X_PNG)) {
     type = DecoderType::PNG;
+  } else if (!strcmp(aMimeType, IMAGE_APNG)) {
+    type = DecoderType::PNG;
 
   // GIF
   } else if (!strcmp(aMimeType, IMAGE_GIF)) {
diff --git a/image/build/nsImageModule.cpp b/image/build/nsImageModule.cpp
@@ -77,6 +77,7 @@ static const mozilla::Module::CategoryEntry kImageCategories[] = {
   { "Gecko-Content-Viewers", IMAGE_BMP_MS, "@mozilla.org/content/document-loader-factory;1" },
   { "Gecko-Content-Viewers", IMAGE_ICON_MS, "@mozilla.org/content/document-loader-factory;1" },
   { "Gecko-Content-Viewers", IMAGE_PNG, "@mozilla.org/content/document-loader-factory;1" },
+  { "Gecko-Content-Viewers", IMAGE_APNG, "@mozilla.org/content/document-loader-factory;1" },
   { "Gecko-Content-Viewers", IMAGE_X_PNG, "@mozilla.org/content/document-loader-factory;1" },
   { "content-sniffing-services", "@mozilla.org/image/loader;1", "@mozilla.org/image/loader;1" },
   { nullptr }
diff --git a/layout/reftests/apng-mime/animated.apng b/layout/reftests/apng-mime/animated.apng
diff --git a/layout/reftests/apng-mime/expected.html b/layout/reftests/apng-mime/expected.html
@@ -0,0 +1,3 @@
+<!doctype html>
+<title>apng expected</title>
+<img src="animated.apng">
diff --git a/layout/reftests/apng-mime/reftest.list b/layout/reftests/apng-mime/reftest.list
@@ -0,0 +1 @@
+== test.html expected.html
diff --git a/layout/reftests/apng-mime/static.png b/layout/reftests/apng-mime/static.png
diff --git a/layout/reftests/apng-mime/test.html b/layout/reftests/apng-mime/test.html
@@ -0,0 +1,6 @@
+<!doctype html>
+<title>apng test</title>
+<picture>
+<source type="video/vnd.mozilla.apng" srcset="animated.apng">
+<img src="static.png">
+</picture>
diff --git a/layout/reftests/reftest.list b/layout/reftests/reftest.list
@@ -19,6 +19,9 @@ include w3c-css/received/reftest.list
 include abs-pos/reftest.list
 include position-relative/reftest.list
 
+# apng-mime
+include apng-mime/reftest.list
+
 include async-scrolling/reftest.list
 
 # backgrounds/
diff --git a/netwerk/mime/nsMimeTypes.h b/netwerk/mime/nsMimeTypes.h
@@ -94,6 +94,7 @@
 #define IMAGE_JPG                           "image/jpg"
 #define IMAGE_PJPEG                         "image/pjpeg"
 #define IMAGE_PNG                           "image/png"
+#define IMAGE_APNG                          "video/vnd.mozilla.apng"
 #define IMAGE_X_PNG                         "image/x-png"
 #define IMAGE_PPM                           "image/x-portable-pixmap"
 #define IMAGE_XBM                           "image/x-xbitmap"
diff --git a/uriloader/exthandler/nsExternalHelperAppService.cpp b/uriloader/exthandler/nsExternalHelperAppService.cpp
@@ -576,6 +576,7 @@ static nsExtraMimeTypeEntry extraMimeEntries [] =
   { IMAGE_ICO, "ico,cur", "ICO Image" },
   { IMAGE_JPEG, "jpeg,jpg,jfif,pjpeg,pjp", "JPEG Image" },
   { IMAGE_PNG, "png", "PNG Image" },
+  { IMAGE_APNG, "apng", "APNG Image" },
   { IMAGE_TIFF, "tiff,tif", "TIFF Image" },
   { IMAGE_XBM, "xbm", "XBM Image" },
   { IMAGE_SVG_XML, "svg", "Scalable Vector Graphics" },

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+<!doctype html>`
	`2`	`+<title>apng expected</title>`
	`3`	`+<img src="animated.apng">`