Commit Graph

899 Commits

Author SHA1 Message Date
zaggy1024
e349b686f8 Bug 1817108 - Check that AVIF frame sizes remain unchanged during animation. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D170193
2023-03-01 07:04:31 +00:00
zaggy1024
0c11da1fd0 Bug 1814708 - Error when the length of AVIF color or alpha sequences mismatch. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D168777
2023-02-07 07:35:32 +00:00
zaggy1024
a55a629ec0 Bug 1814561 - Error out when samples fail to be read from an AVIF sequence. r=tnikkel
The iterator used to read from an AVIF sample index will return `true` from
`HasNext()` even if the next sample is incomplete. We should error completely
out of the decode when this happens, so that we are always guaranteed to have a
sample to decode when there is no error in the parser.

Differential Revision: https://phabricator.services.mozilla.com/D168643
2023-02-07 07:28:55 +00:00
zaggy1024
5117018c2f Bug 1814553 - Make sure to always send AVIF decoder error telemetry. r=tnikkel
This fixes an assertion failure due to the assumption before Bug 1682662 that
all metadata decodes will completely decode a sample. That is no longer the case
because telemetry for image sizes can be sent without a full decode.

Differential Revision: https://phabricator.services.mozilla.com/D168645
2023-02-07 06:34:57 +00:00
zaggy1024
2697cbe267 Bug 1814677 - Use the decoded image size for SurfacePipe frame rectangle. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D168773
2023-02-07 05:55:17 +00:00
zaggy1024
3fcaa79602 Bug 1814741 - Store AVIFDecoderStream in a RefPtr. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D168793
2023-02-06 02:13:37 +00:00
zaggy1024
8f9447e7ed Bug 1814774 - Error early when a different-sized AVIF alpha plane is decoded. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D168892
2023-02-04 13:17:22 +00:00
zaggy1024
e5729af5b8 Bug 1810613 - Part 2 - Apply loop counts from AVIF animation files. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D168345
2023-02-01 09:45:41 +00:00
zaggy1024
72068f13d2 Bug 1810613 - Part 1 - Update mp4parse to 40763b6. r=kinetik
Differential Revision: https://phabricator.services.mozilla.com/D168344
2023-02-01 09:45:40 +00:00
Stephen A Pohl
bc29fba3c6 Bug 1779478: Replace the deprecated use of NSGraphicsContext's graphicsPort with CGContext. r=mstange
Depends on D153637

Differential Revision: https://phabricator.services.mozilla.com/D153639
2023-01-31 15:50:00 +00:00
zaggy1024
3dc9e90e9a Bug 1788119 - Part 3 - Add initial support for animated AVIF sequences. r=tnikkel
Depends on D156651

Differential Revision: https://phabricator.services.mozilla.com/D156652
2023-01-27 21:25:50 +00:00
zaggy1024
8d70b2d53e Bug 1788119 - Part 1 - Keep parser and decoder instances persistent in nsAVIFDecoder. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D156650
2023-01-27 21:25:48 +00:00
Butkovits Atila
2e5eede2fe Backed out 5 changesets (bug 1788119) for causing bustages complaining about AVIFDecodedData. CLOSED TREE
Backed out changeset 430e71478ff5 (bug 1788119)
Backed out changeset 9493029c76b9 (bug 1788119)
Backed out changeset 7d5cc211e3f5 (bug 1788119)
Backed out changeset 3eaac75dde3c (bug 1788119)
Backed out changeset 1f8eacf96c71 (bug 1788119)
2023-01-27 22:54:08 +02:00
zaggy1024
5c801c6e9e Bug 1788119 - Part 3 - Add initial support for animated AVIF sequences. r=tnikkel
Depends on D156651

Differential Revision: https://phabricator.services.mozilla.com/D156652
2023-01-27 19:54:29 +00:00
zaggy1024
bb345c73b5 Bug 1788119 - Part 1 - Keep parser and decoder instances persistent in nsAVIFDecoder. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D156650
2023-01-27 19:54:28 +00:00
Norisz Fay
58c65f7c8f Backed out 5 changesets (bug 1788119) for causing build bustages on DecoderData.cpp CLOSED TREE
Backed out changeset 56abc66f7e84 (bug 1788119)
Backed out changeset 00d82acfaa2f (bug 1788119)
Backed out changeset 4ba88410cdf6 (bug 1788119)
Backed out changeset eeb1461933aa (bug 1788119)
Backed out changeset 8dce9350abb8 (bug 1788119)
2023-01-27 21:11:42 +02:00
zaggy1024
a50faf8760 Bug 1788119 - Part 3 - Add initial support for animated AVIF sequences. r=tnikkel
Depends on D156651

Differential Revision: https://phabricator.services.mozilla.com/D156652
2023-01-27 18:35:14 +00:00
zaggy1024
13ebc5f3b4 Bug 1788119 - Part 1 - Keep parser and decoder instances persistent in nsAVIFDecoder. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D156650
2023-01-27 18:35:13 +00:00
Timothy Nikkel
f9d4662dba Bug 1801397. Render questionable 1x1 gifs with no image data like other clients. r=aosmond
Differential Revision: https://phabricator.services.mozilla.com/D162450
2022-12-15 00:41:56 +00:00
Nika Layzell
1f06e7ba7b Bug 1793463 - Part 6: Remove nsIProtocolHandler.{defaultPort,protocolFlags}, r=necko-reviewers,valentin
This removes all implementations of these types. Some implementations in JS
code were also removed in the previous part, when updating tests to use
`Services.io.registerProtocolHandler`.

Code which used to access these members should go through the IOService now
instead.

Differential Revision: https://phabricator.services.mozilla.com/D162805
2022-12-01 15:43:19 +00:00
Nika Layzell
bca0a6965b Bug 1793463 - Part 4: Generate static components entries for protocols, r=necko-reviewers,xpcom-reviewers,valentin,kmag
This adds a new set of options to static components.conf files to allow
specifying the protocol flags and default ports of a protocol handler, and
generates a separate table just for this purpose.

This will be used in the next part as part of replacing the existing protocol
handler lookup infrastructure.

Differential Revision: https://phabricator.services.mozilla.com/D162803
2022-12-01 15:43:19 +00:00
Marian-Vasile Laza
7492025127 Backed out 7 changesets (bug 1793463) for causing bustages on nsIOService.cpp. CLOSED TREE
Backed out changeset 47c3acb30de2 (bug 1793463)
Backed out changeset 71afe900eb17 (bug 1793463)
Backed out changeset d168599a269e (bug 1793463)
Backed out changeset d0f18d300818 (bug 1793463)
Backed out changeset a116e6f089f2 (bug 1793463)
Backed out changeset eba1123c8b56 (bug 1793463)
Backed out changeset 68a3eb8a3057 (bug 1793463)
2022-12-01 02:30:02 +02:00
Nika Layzell
20db36cecb Bug 1793463 - Part 6: Remove nsIProtocolHandler.{defaultPort,protocolFlags}, r=necko-reviewers,valentin
This removes all implementations of these types. Some implementations in JS
code were also removed in the previous part, when updating tests to use
`Services.io.registerProtocolHandler`.

Code which used to access these members should go through the IOService now
instead.

Differential Revision: https://phabricator.services.mozilla.com/D162805
2022-11-30 18:13:43 +00:00
Nika Layzell
90f7562d95 Bug 1793463 - Part 4: Generate static components entries for protocols, r=necko-reviewers,xpcom-reviewers,valentin,kmag
This adds a new set of options to static components.conf files to allow
specifying the protocol flags and default ports of a protocol handler, and
generates a separate table just for this purpose.

This will be used in the next part as part of replacing the existing protocol
handler lookup infrastructure.

Differential Revision: https://phabricator.services.mozilla.com/D162803
2022-11-30 18:13:42 +00:00
Cristian Tuns
dee91a94c7 Backed out changeset 791d453288b4 (bug 1801397) for causing gtest failures with ImageDecoders CLOSED TREE 2022-11-29 01:20:08 -05:00
Timothy Nikkel
8df39133e8 Bug 1801397. Render questionable 1x1 gifs with no image data like other clients. r=aosmond
Differential Revision: https://phabricator.services.mozilla.com/D162450
2022-11-29 05:35:49 +00:00
Daniel Holbert
7f83cbcfba Bug 1800342: Make NS_NewPipe2 and NS_NewPipe infallible, to reflect reality. r=xpcom-reviewers,necko-reviewers,nika,kershaw
This patch doesn't change behavior; these APIs were already effectively
infallible, in the sense that it was impossible for them to return anything
other than NS_OK.

The actual API changes here are in xpcom/io/nsIPipe.idl and
xpcom/io/nsPipe3.cpp, and the rest of this patch is just updates to callsites,
removing code that handled failure-flavored return values from these functions.
(All of this removed code was already dead code, effectively.)

Differential Revision: https://phabricator.services.mozilla.com/D161947
2022-11-14 23:22:16 +00:00
Brad Werth
987a61fc5c Bug 1798532 Part 3: Allow JPEGDecoder to handle frame data in the footer. r=tnikkel
This also modifies red-bad-marker.jpg to add a SOF6 marker (0xFFC6) in the
footer.

Differential Revision: https://phabricator.services.mozilla.com/D161637
2022-11-09 17:01:35 +00:00
Brad Werth
cbb5bf5c93 Bug 1798532 Part 1: Ignore JPEG unknown markers. r=tnikkel
This will push through any unknown marker errors from libjpeg, but still
fail for other errors. Dropping markers prevents infinite loops in processing.

Differential Revision: https://phabricator.services.mozilla.com/D161187
2022-11-07 22:20:36 +00:00
Timothy Nikkel
19b7441082 Bug 1794754. Return NEED_MORE_DATA from nsWebPDecoder::ReadHeader if ICC was specified but chunk not found r=aosmond
Bug 1780834 caused this bug. If we are not complete then we might need to
wait for more data to come in that has the ICCP chunk.

Differential Revision: https://phabricator.services.mozilla.com/D159137
2022-10-20 07:53:25 +00:00
Masatoshi Kimura
a6888b17a9 Bug 327074 - Allow RLE bitmaps with extra pixels per row to render. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D159028
2022-10-12 03:58:32 +00:00
Brad Werth
949fa28f4f Bug 1765187 Part 1: Track color primaries independently from coefficients. r=media-playback-reviewers,jgilbert,alwu
This patch attempts to read color primary information from platform
agnostic decoders. It doesn't use any of this information to change how
the video frame is displayed.

It also cleans up some VPX transfer characteristics, to ensure they are
actually retrieved from the codec information, when available.

Differential Revision: https://phabricator.services.mozilla.com/D156362
2022-10-10 20:12:58 +00:00
Chun-Min Chang
c3449a3361 Bug 1774300 - Implement VideoFrame Constructor for I420A ArrayBuffer* r=padenot,jgilbert
This patch allows constructing a VideoFrame from the ArrayBuffer* in
I420A format, which contains the I420 data with an extra alpha channel
data.

Depends on D149584

Differential Revision: https://phabricator.services.mozilla.com/D149943
2022-10-06 00:37:22 +00:00
Butkovits Atila
dd8f948670 Backed out 19 changesets (bug 1793497, bug 1774300) for causing lint failure. CLOSED TREE
Backed out changeset f5460317d244 (bug 1793497)
Backed out changeset 5af8771526a1 (bug 1774300)
Backed out changeset 3ab7aead37b5 (bug 1774300)
Backed out changeset 5f837d7e621b (bug 1774300)
Backed out changeset 56bfa6480cb5 (bug 1774300)
Backed out changeset db37fffc8519 (bug 1774300)
Backed out changeset d5d263cedbc9 (bug 1774300)
Backed out changeset 52fdd5a613f4 (bug 1774300)
Backed out changeset 6ce76e0d6fd2 (bug 1774300)
Backed out changeset c5f67f22597e (bug 1774300)
Backed out changeset 4a7dca31703e (bug 1774300)
Backed out changeset e56596b91a86 (bug 1774300)
Backed out changeset 825ae6e7be9b (bug 1774300)
Backed out changeset 19f750d86a73 (bug 1774300)
Backed out changeset 50995e55c546 (bug 1774300)
Backed out changeset 33b150b4c911 (bug 1774300)
Backed out changeset 2bc06d0e8808 (bug 1774300)
Backed out changeset ba9bc5c631b0 (bug 1774300)
Backed out changeset 4a145bdd6b86 (bug 1774300)
2022-10-05 01:05:57 +03:00
Chun-Min Chang
c4b771507e Bug 1774300 - Implement VideoFrame Constructor for I420A ArrayBuffer* r=padenot,jgilbert
This patch allows constructing a VideoFrame from the ArrayBuffer* in
I420A format, which contains the I420 data with an extra alpha channel
data.

Depends on D149584

Differential Revision: https://phabricator.services.mozilla.com/D149943
2022-10-04 21:38:26 +00:00
Andrew McCreight
2da84b8ac4 Bug 1792574, part 3 - Don't include nsMemory where it isn't needed. r=xpcom-reviewers,necko-reviewers,valentin,nika
There are only 3 places where nsMemory.h is still needed (image/RasterImage.cpp,
gfx/thebes/gfxFT2FontList.cpp, and nsMemory.cpp). Remove the rest.

Differential Revision: https://phabricator.services.mozilla.com/D158213
2022-09-28 15:17:46 +00:00
Dana Keeler
923d7b2029 Bug 1790451 - make nsIChannel.securityInfo concrete as nsITransportSecurityInfo r=kershaw,necko-reviewers,valentin,geckoview-reviewers,owlish
Differential Revision: https://phabricator.services.mozilla.com/D157136
2022-09-20 03:58:49 +00:00
Kershaw Chang
cc6c147f88 Bug 1760580 - P1: Make nsIRequest support to store canceled reason, r=necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D156070
2022-09-08 09:11:18 +00:00
Andi-Bogdan Postelnicu
eeac3728d2 Bug 1519636 - Reformat recent changes to the Google coding style. r=glandium
Updated with clang-format version 14.0.5 (taskcluster-DydCt-ryTuKvBYw1HQOugw)
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D154661
2022-08-16 07:08:27 +00:00
Mike Hommey
99d04c97e8 Bug 1784205 - Avoid returning SIID_INVALID as a SHSTOCKICONID because it's not part of the enum. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D154355
2022-08-11 08:26:53 +00:00
Timothy Nikkel
b5a9365200 Bug 1780834. If a webp file header specifies ICCP but no ICCP chunk is found just ignore it and continue. r=aosmond
Differential Revision: https://phabricator.services.mozilla.com/D152588
2022-07-26 08:24:27 +00:00
Dragana Damjanovic
0cb9ae544a Bug 1778272 - Revert the change to the LoadInfo release function. r=necko-reviewers,kershaw
Differential Revision: https://phabricator.services.mozilla.com/D151103
2022-07-06 08:10:51 +00:00
Emilio Cobos Álvarez
3edae2885b Bug 1757929 - Avoid some conflicts with WINDOWS define. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D145569
2022-05-05 11:37:51 +00:00
Andrew Osmond
d70a78efc1 Bug 1758975 - Ensure GIF color tables are opaque by default. r=tnikkel
Differential Revision: https://phabricator.services.mozilla.com/D141176
2022-03-18 15:22:46 +00:00
Lee Salzman
6f0be8c19f Bug 1757067 - Explicitly specify chroma subsampling scheme for YCbCr data. r=jgilbert,jrmuizel
Differential Revision: https://phabricator.services.mozilla.com/D139656
2022-03-10 09:24:15 +00:00
smolnar
97f77f8381 Backed out changeset 36c6350bc7d1 (bug 1757067) for causing build bustages in gfx/gl/GLBlitHelperD3D.cpp CLOSED TREE 2022-03-10 10:10:10 +02:00
Lee Salzman
a664a6d305 Bug 1757067 - Explicitly specify chroma subsampling scheme for YCbCr data. r=jgilbert,jrmuizel
Differential Revision: https://phabricator.services.mozilla.com/D139656
2022-03-10 07:48:53 +00:00
Nicklas Boman
368311a741 Bug 1725363 - Replace PL_strcasecmp with nsCRT::strcasecmp image/ r=aosmond
Differential Revision: https://phabricator.services.mozilla.com/D129074
2022-01-26 09:07:33 +00:00
Mike Hommey
fce6eb7d83 Bug 1747165 - Replace TK_FLAGS/TK_LIBS with MOZ_GTK3_FLAGS/MOZ_GTK3_LIBS. r=firefox-build-system-reviewers,andi
Differential Revision: https://phabricator.services.mozilla.com/D134464
2021-12-23 20:29:07 +00:00
Cristian Tuns
e1e31e01f2 Backed out 2 changesets (bug 1747165) for causing nightly blockers(bustages) a=backout
Backed out changeset 4b1ab0915c94 (bug 1747165)
Backed out changeset 96043d814772 (bug 1747165)
2021-12-23 08:00:54 -05:00