Skip to content

GitLab

T
tangled
Commit 61d26e9d authored by Fredrik Söderquist's avatar Fredrik Söderquist Committed by Commit Bot
Browse files
Options

[PE] Add "image/apng" to supported image MIME types 

Supported for animated PNG (APNG) was added via crbug.com/437662 (commit
7d2b8c45.) The "image/apng" MIME type is
included in the "Accept" header for image requests.

Add "image/apng" to kSupportedImageTypes to allow things like the
<picture> element to work (select an appropriate image based on 'type'.)

Add "image/apng" to kPrimaryMappings to allow the MIME type to resolve
to a reasonable file extension.

Bug: 730588
Change-Id: I070e8b16dc8312552b11dc32e776493b3a5be669
Reviewed-on: https://chromium-review.googlesource.com/957092
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Reviewed-by: default avatarAsanka Herath <asanka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542113}
parent e198a121
Hide whitespace changes
Inline Side-by-side
Showing with 25 additions and 0 deletions
net/base/mime_util.cc
View file @ 61d26e9d
...@@ -92,6 +92,7 @@ static const MimeInfo kPrimaryMappings[] = { ...@@ -92,6 +92,7 @@ static const MimeInfo kPrimaryMappings[] = {
{"image/gif", "gif"}, {"image/gif", "gif"},
{"image/jpeg", "jpeg,jpg"}, {"image/jpeg", "jpeg,jpg"},
{"image/png", "png"}, {"image/png", "png"},
{"image/apng", "png"},
{"image/webp", "webp"}, {"image/webp", "webp"},
{"multipart/related", "mht,mhtml"}, {"multipart/related", "mht,mhtml"},
{"text/css", "css"}, {"text/css", "css"},
......
third_party/WebKit/LayoutTests/external/wpt/apng/supported-in-source-type.html 0 → 100644
View file @ 61d26e9d
<!DOCTYPE html>
<title>Animated PNG MIME type (image/apng) is recognized by &lt;source type></title>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<script>
function resolveUrl(relativeUrl) {
var a = document.createElement('a');
a.href = relativeUrl;
return a.href;
}
async_test(t => {
window.onload = t.step_func_done(() => {
let image = document.querySelector('img');
let apngSrc = document.querySelector('source');
assert_equals(image.currentSrc, resolveUrl(apngSrc.srcset));
});
});
</script>
<picture>
<source srcset="/images/anim-gr.png" type="image/apng">
<img src="/images/anim-gr.gif" style="visibility: hidden">
</picture>
third_party/WebKit/common/mime_util/mime_util.cc
View file @ 61d26e9d
...@@ -29,6 +29,7 @@ const char* const kSupportedImageTypes[] = {"image/jpeg", ...@@ -29,6 +29,7 @@ const char* const kSupportedImageTypes[] = {"image/jpeg",
"image/jpg", "image/jpg",
"image/webp", "image/webp",
"image/png", "image/png",
"image/apng",
"image/gif", "image/gif",
"image/bmp", "image/bmp",
"image/vnd.microsoft.icon", // ico "image/vnd.microsoft.icon", // ico
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment