Andrew Osmond
9d456f9fe3
Bug 1293577 - Protect the image decoding task path with a mutex to avoid race conditions. r=seth
2016-08-10 07:35:07 -04:00
Nicholas Nethercote
e7f10a07fd
Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm.
...
This patch makes most Run() declarations in subclasses of nsIRunnable have the
same form: |NS_IMETHOD Run() override|.
As a result of these changes, I had to add |override| to a couple of other
functions to satisfy clang's -Winconsistent-missing-override warning.
--HG--
extra : rebase_source : 815d0018b0b13329bb5698c410f500dddcc3ee12
2016-08-08 12:18:10 +10:00
Seth Fowler
ba8bb5de9d
Bug 1292747 (Part 3) - Clean up some comments in imgFrame related to Optimize()'s behavior. r=edwin
2016-08-08 16:25:50 -07:00
Seth Fowler
ad2f26e773
Bug 1292747 (Part 2) - Remove the 565 optimization from imgFrame. r=edwin
2016-08-08 16:25:48 -07:00
Seth Fowler
55d13bdddf
Bug 1292747 (Part 1) - Remove 565 optimization telemetry. r=edwin
2016-08-08 16:25:46 -07:00
Jonathan Hao
ea61afe035
Bug 1270680 - Part 1: Double-key the image cache by origin attribute. r=jdm
2016-08-04 20:22:00 +02:00
Seth Fowler
ec7c520520
Bug 1291071 (Part 6) - Stop passing a decoder to FinalizeDecoder() and rename it NotifyDecodeComplete(). r=edwin
2016-08-05 18:50:32 -07:00
Seth Fowler
0403b4baba
Bug 1291071 (Part 5) - Pass the decoder's final status explicitly to FinalizeDecoder(). r=edwin
2016-08-05 18:50:31 -07:00
Seth Fowler
98a9fd7db6
Bug 1291071 (Part 4) - Clean up Decoder::SpeedHistogram() and related code. r=edwin
2016-08-05 18:50:29 -07:00
Seth Fowler
baf4983cd8
Bug 1291071 (Part 3) - Pass telemetry explicitly to FinalizeDecoder. r=edwin
2016-08-05 18:50:27 -07:00
Seth Fowler
e3510bdeff
Bug 1291071 (Part 2) - Pass decoder progress explicitly to FinalizeDecoder. r=edwin
2016-08-05 18:50:25 -07:00
Seth Fowler
b765419db3
Bug 1291071 (Part 1) - Pass ImageMetadata explicitly to FinalizeDecoder. r=edwin
2016-08-05 18:50:24 -07:00
Seth Fowler
5eeda4dad6
Bug 1291071 (Part 1) - Move an assertion from RasterImage::FinalizeDecoder() to IDecodingTask::NotifyDecodeComplete(). r=edwin
2016-08-05 18:50:22 -07:00
Seth Fowler
f5495f0fd1
Bug 1291045 (Part 7) - Replace DecodingTask with DecodedSurfaceProvider. r=dholbert,edwin
2016-08-05 18:46:24 -07:00
Seth Fowler
1cd0225e12
Bug 1291045 (Part 6) - Add DecodedSurfaceProvider to handle both decoding and surface ownership for single-frame images. r=dholbert,edwin
2016-08-05 18:46:22 -07:00
Seth Fowler
1324b84fef
Bug 1291045 (Part 5) - Expose the IDecodingTask notification helpers for use in other files. r=edwin
2016-08-05 18:46:19 -07:00
Seth Fowler
981466ae1a
Bug 1291045 (Part 4) - Remove IDecodingTask::GetDecoder(). r=edwin
2016-08-05 18:46:16 -07:00
Seth Fowler
694b00bcd2
Bug 1291045 (Part 3) - Handle interactions with the SurfaceCache in DecodingTask. r=dholbert,edwin
2016-08-05 18:46:13 -07:00
Seth Fowler
ba6bdbca1c
Bug 1291045 (Part 2) - Pass the image into NotifyProgress() and NotifyDecodeComplete() explicitly. r=edwin
2016-08-05 18:46:12 -07:00
Seth Fowler
a604246bfc
Bug 1291045 (Part 1) - Use a different IDecodingTask for animated images. r=dholbert,edwin
2016-08-05 18:46:10 -07:00
Daniel Holbert
2e355bc011
Bug 1292632 part 3: Include <algorithm> in image/SourceBuffer.h, to provide std::min. r=seth
...
MozReview-Commit-ID: DJvb7Sz4efF
2016-08-05 17:09:48 -07:00
Daniel Holbert
19d78aab9b
Bug 1292632 part 2: Add missing "using" decl for IntRect in IDecodingTask.cpp. r=seth
...
MozReview-Commit-ID: 6OeckqKu7k7
2016-08-05 17:09:47 -07:00
Daniel Holbert
ac90698e93
Bug 1292632 part 1: Remove a stale/unused #include from SurfaceCache.cpp. r=seth
...
MozReview-Commit-ID: feZgLQ1R7u
2016-08-05 17:09:45 -07:00
Seth Fowler
cf5f340405
Bug 1292505 (Followup) - Add an assert that we don't create a decoder for an ICO resource during a metadata decode. r=me
2016-08-05 15:04:20 -07:00
Seth Fowler
19dc4aba32
Bug 1292505 (Part 2) - Remove remaining references to Downscaler in the SurfacePipe decoders. r=edwin
2016-08-05 13:44:52 -07:00
Seth Fowler
6a44833af8
Bug 1292505 (Part 1c) - Use ExplicitOutputSize() instead of Downscaler::TargetSize() in nsICODecoder. r=edwin
2016-08-05 13:44:48 -07:00
Seth Fowler
1e04c585b2
Bug 1292505 (Part 1b) - Update DecoderFactory to use SetOutputSize(), and propagate the changes to RasterImage. r=edwin
2016-08-05 13:44:46 -07:00
Seth Fowler
f2c6e25053
Bug 1292505 (Part 1a) - Replace Decoder::SetTargetSize() with Decoder::SetOutputSize(). r=edwin
2016-08-05 13:44:44 -07:00
Igor
4efade90d4
Bug 1197331 - remove PR_snprintf calls in miscellaneous directories. r=nfroyd
...
--HG--
extra : rebase_source : a19f77db06e68bd3d69faf56b8bbdc1825f13249
2016-08-04 16:41:00 +02:00
Yoshi Huang
a520e75a4d
Bug 1264231 - Part 3: remove loadInfo work-around. r=sicking
2016-08-05 19:44:57 +08:00
Seth Fowler
2b78b14657
Bug 1291033 (Part 2) - Ensure atomicity of ISurfaceProvider locking changes. r=dholbert
2016-08-04 18:47:43 -07:00
Seth Fowler
df8d9955b6
Bug 1291033 (Part 1) - Ensure atomicity of ISurfaceProvider availability changes. r=dholbert
2016-08-04 18:47:41 -07:00
Carsten "Tomcat" Book
389a3e0817
merge mozilla-inbound to mozilla-central a=merge
...
--HG--
rename : mobile/android/base/java/org/mozilla/gecko/GeckoAppShell.java => mobile/android/geckoview/src/main/java/org/mozilla/gecko/GeckoAppShell.java
rename : mobile/android/base/java/org/mozilla/gecko/gfx/GeckoLayerClient.java => mobile/android/geckoview/src/main/java/org/mozilla/gecko/gfx/GeckoLayerClient.java
rename : mobile/android/base/java/org/mozilla/gecko/gfx/LayerRenderer.java => mobile/android/geckoview/src/main/java/org/mozilla/gecko/gfx/LayerRenderer.java
rename : mobile/android/base/java/org/mozilla/gecko/gfx/PanningPerfAPI.java => mobile/android/geckoview/src/main/java/org/mozilla/gecko/gfx/PanningPerfAPI.java
2016-08-04 15:55:50 +02:00
Alexandre Lissy
0af5b943b6
Bug 1284674 - Remove NUWA r=cyu
...
MozReview-Commit-ID: GyMRNzOBKw6
--HG--
extra : rebase_source : 293af1cd55f2035ce6a99f4ebf144059c32a2b8f
2016-08-02 14:54:00 +02:00
Seth Fowler
9f0a5862e4
Bug 1291059 - Yield in the GIF decoder at the beginning of a new frame, not at the end of the previous frame. r=edwin
2016-08-02 04:10:59 -07:00
Seth Fowler
bef1e68a73
Bug 1291054 (Part 5) - Remove Decoder::AllocateBasicFrame(), which is dead code. r=edwin
2016-08-02 04:10:56 -07:00
Seth Fowler
6ce660afb2
Bug 1291054 (Part 4) - Add Decoder convenience methods for the common case of frame rects that cover the whole image. r=edwin
2016-08-02 04:10:52 -07:00
Seth Fowler
3cca5ff418
Bug 1291054 (Part 3) - Rename Decoder::GetSize() to Decoder::Size() to be consistent with the style guide. r=edwin
2016-08-02 04:10:51 -07:00
Seth Fowler
881c87ebec
Bug 1291054 (Part 2) - Add a Decoder::OutputSize() getter and use it in the decoders. r=edwin
2016-08-02 04:10:49 -07:00
Seth Fowler
1d1a71d4bc
Bug 1291054 (Part 1) - Clean up Decoder's size getters a little bit. r=edwin
2016-08-02 04:10:47 -07:00
Glenn Randers-Pehrson
a5af07b18b
Bug 1240665 - Log all error returns from the PNG decoder. r=tnikkel
...
--HG--
extra : rebase_source : 373eef390fe03a2559e336539ddc6a546d6ec2d7
2016-07-29 05:11:00 +02:00
Seth Fowler
2799864a33
Bug 1290682 (Part 3) - Remove RasterImage::CopyFrame(). r=edwin
2016-08-01 17:51:20 -07:00
Seth Fowler
f09a1b3d95
Bug 1290682 (Part 2) - Clean up imgFrame::Draw() and related code now that we know no non-trivial frame rects will occur. r=edwin
2016-08-01 17:51:18 -07:00
Seth Fowler
94cd474a67
Bug 1290682 (Part 1) - Explicitly forbid non-trivial frame rects in imgFrame::Draw() except for paletted frames. r=edwin
2016-08-01 17:51:16 -07:00
Seth Fowler
e36fc95869
Bug 1290681 (Part 2) - Remove the single color optimization from imgFrame. r=edwin
2016-08-01 17:51:13 -07:00
Seth Fowler
b704f1ab3d
Bug 1290681 (Part 1) - Remove telemetry for the single color optimization. r=edwin
2016-08-01 17:51:11 -07:00
Seth Fowler
4d73b5b868
(No bug) - Improve comment in IDecodingTask.cpp. r=me
2016-08-01 17:51:04 -07:00
Seth Fowler
16cff1aefa
Bug 1290759 - Convert frame number check in Decoder::AllocateFrame() into an assert. r=edwin
2016-08-01 17:40:05 -07:00
Seth Fowler
9371310b88
Bug 1290747 - Fix bad assert in RasterImage::NotifyProgress(). r=edwin
2016-08-01 17:40:03 -07:00
Glenn Randers-Pehrson
e2b623ebe9
Bug 1283961 - Part 2: Remove limits on PNG image dimensions (nsPNGDecoder). r=seth
...
--HG--
extra : rebase_source : 8c138a12011233e320d4e55e95348c4c961b95b6
2016-07-28 07:37:00 -04:00
Seth Fowler
97b4a5d8c7
Bug 1289957 (Part 2) - Notify RasterImage about new frames in NotifyProgress() and remove OnAddedFrame(). r=edwin
2016-07-28 00:12:50 -07:00
Seth Fowler
eb5bac10fc
Bug 1289957 (Part 1) - Fix minor style issues in RasterImage::NotifyProgress. r=edwin
2016-07-28 00:12:49 -07:00
Andrew Osmond
0dc52b5140
Bug 1283462 - Fix race condition when notifying on image decoding progress. r=seth
2016-07-22 11:04:09 -04:00
Seth Fowler
77c1dbccdb
Bug 1289628 - Return ISurfaceProvider objects from SurfaceCache lookup functions. r=dholbert,edwin
2016-07-27 13:54:23 -07:00
Gijs Kruitbosch
61094e5a36
Bug 1280584 - implement cloneWithNewRef and thereby make hash/ref links use a simple unified codepath in the IO service, r=valentin
...
MozReview-Commit-ID: 8FIyclkJPDp
--HG--
extra : rebase_source : 49fb2b12488bd57a5417c7c3bff2211d651c5de7
extra : amend_source : d9940a1d9009930dc2f499417a7e1446d4601c20
2016-07-26 23:38:46 +01:00
Nicolas Silva
48805c9b3e
Bug 1284837 - Disallow implicit conversions from float to integer when creating IntPoint and IntSize objects. r=botond
2016-07-26 16:48:30 +02:00
Carsten "Tomcat" Book
336105a0de
merge mozilla-inbound to mozilla-central a=merge
2016-07-22 11:58:02 +02:00
Seth Fowler
0f56cc072c
(No bug) - Fix a trivial formatting issue in ImageMetadata.h. r=me
2016-07-21 16:42:29 -07:00
Tom Tromey
5538d692d3
Bug 1286877 - do not set c-basic-offset for python-mode; r=gps
...
This removes the unnecessary setting of c-basic-offset from all
python-mode files.
This was automatically generated using
perl -pi -e 's/; *c-basic-offset: *[0-9]+//'
... on the affected files.
The bulk of these files are moz.build files but there a few others as
well.
MozReview-Commit-ID: 2pPf3DEiZqx
--HG--
extra : rebase_source : 0a7dcac80b924174a2c429b093791148ea6ac204
2016-07-14 10:16:42 -06:00
Chris Peterson
b175c9fdd5
Bug 1277106 - Part 2: Expand MOZ_UTF16() strings to u"" string literals. r=Waldo
2016-07-20 22:03:25 -07:00
Seth Fowler
dc9bddc5cd
Bug 1288040 (Part 12) - Use Moz2D types in FrameAnimator code. r=edwin
2016-07-20 16:30:44 -07:00
Seth Fowler
8bb93146a8
Bug 1288040 (Part 11) - Clean up RefreshResult. r=edwin
2016-07-20 16:30:41 -07:00
Seth Fowler
3e8d8d790f
Bug 1288040 (Part 10) - Determine the first frame refresh area of animated images while decoding them. r=edwin
2016-07-20 16:30:39 -07:00
Seth Fowler
642c28d837
Bug 1288040 (Part 9) - Determine the loop length of animated images while decoding them. r=edwin
2016-07-20 16:30:36 -07:00
Seth Fowler
049339870a
Bug 1288040 (Part 8) - Return a FrameTimeout value from FrameAnimator::GetSingleLoopTime(). r=edwin
2016-07-20 16:30:34 -07:00
Seth Fowler
930f26424f
Bug 1288040 (Part 7) - Make FrameAnimator::GetTimeoutForFrame() a private method that doesn't rely on AnimationState. r=edwin
2016-07-20 16:30:32 -07:00
Seth Fowler
aa9b084d09
Bug 1288040 (Part 6) - Don't call GetTimeoutForFrame() in RasterImage. r=edwin
2016-07-20 16:30:30 -07:00
Seth Fowler
78ccdc2e84
Bug 1288040 (Part 5) - Wrap frame timeout values in a FrameTimeout type that ensures they're normalized. r=edwin
2016-07-20 16:30:28 -07:00
Seth Fowler
fc0b1f8b27
Bug 1288040 (Part 4) - We only need to check for infinitely long frames in StartAnimation() for the first frame. r=edwin
2016-07-20 16:30:26 -07:00
Seth Fowler
4306ec9de5
Bug 1288040 (Part 3) - Get rid of RefreshResult.error, a field which nothing cares about. r=edwin
2016-07-20 16:30:24 -07:00
Seth Fowler
4af805ced1
Bug 1288040 (Part 2) - Don't reset the last composited frame index when we reset animation. r=edwin
2016-07-20 16:30:21 -07:00
Seth Fowler
34caf5552c
Bug 1288040 (Part 1) - Separate FrameAnimator's state into a separate class, AnimationState. r=edwin
2016-07-20 16:30:19 -07:00
Seth Fowler
a0b0263e26
Bug 1287691 (Part 4) - Leave notifying decoding progress for each frame to DecodingTask. r=edwin
2016-07-20 02:30:39 -07:00
Seth Fowler
62bf80f260
Bug 1287691 (Part 3) - Yield after each frame in the PNG decoder. r=edwin
2016-07-20 02:30:37 -07:00
Seth Fowler
6d790d3506
Bug 1287691 (Part 2) - Yield after each frame in the GIF decoder. r=edwin
2016-07-20 02:30:35 -07:00
Seth Fowler
7edd4cdb0d
Bug 1287691 (Part 1) - Expose yielding to decoding tasks. r=edwin
2016-07-20 02:30:33 -07:00
Makoto Kato
1cb4228397
Backed out 4 changesets (bug 1287691) due to Windows build failure
...
MozReview-Commit-ID: KgV3nU9a64F
2016-07-20 11:03:59 +09:00
Seth Fowler
c4e69d01bb
Bug 1287691 (Part 4) - Leave notifying decoding progress for each frame to DecodingTask. r=edwin
2016-07-19 17:15:01 -07:00
Seth Fowler
78f3ee9ee6
Bug 1287691 (Part 3) - Yield after each frame in the PNG decoder. r=edwin
2016-07-19 17:15:01 -07:00
Seth Fowler
014dc44721
Bug 1287691 (Part 2) - Yield after each frame in the GIF decoder. r=edwin
2016-07-19 17:15:01 -07:00
Seth Fowler
966d0547b7
Bug 1287691 (Part 1) - Expose yielding to decoding tasks. r=edwin
2016-07-19 17:15:01 -07:00
Seth Fowler
666fc943cb
Bug 1287367 - Allow users of StreamingLexer to detect and handle truncation. r=njn
2016-07-18 23:41:20 -07:00
Seth Fowler
b4d3a89f33
Bug 1287246 (Part 5) - Add GTests for StreamingLexer's support for yielding. r=njn
2016-07-18 01:25:45 -07:00
Seth Fowler
fabd6521fe
Bug 1287246 (Part 4) - Make the data checking in TestStreamingLexer more discerning. r=njn
2016-07-18 01:25:43 -07:00
Seth Fowler
d2fa2b335c
Bug 1287246 (Part 3) - Add support for yielding to StreamingLexer. r=njn
2016-07-18 01:25:41 -07:00
Seth Fowler
f5ecac110e
Bug 1287246 (Part 2) - Store the state for unbuffered reads in a separate structure in StreamingLexer. r=njn
2016-07-18 01:25:39 -07:00
Seth Fowler
938a963f7f
Bug 1287246 (Part 1) - Expose LexerResult from the StreamingLexer API and add an explicit Yield type. r=njn
2016-07-18 01:25:37 -07:00
Seth Fowler
342a16f1a3
Bug 1284031 (Part 3) - Remove the FromSourceBuffer suffix from StreamingLexer test names. r=njn
2016-07-16 00:37:06 -07:00
Seth Fowler
71a5e7a304
Bug 1284031 (Part 2) - Add new StreamingLexer tests for zero-length states. r=njn
2016-07-16 00:37:04 -07:00
Seth Fowler
269c03d80e
Bug 1284031 (Part 1) - Advance SourceBufferIterator in Lex() per-state. r=njn
2016-07-16 00:37:02 -07:00
Seth Fowler
6fbb8890f0
Bug 1286161 (Part 2) - Add a GTest suite for SourceBuffer. r=edwin
2016-07-16 00:37:00 -07:00
Seth Fowler
6b52f4548a
Bug 1286161 (Part 1) - Expose SourceBuffer and SurfaceCache constants which are useful for testing. r=edwin
2016-07-16 00:36:58 -07:00
Seth Fowler
af70a29095
Bug 1286175 - Allow SourceBufferIterator users to specify how many bytes to advance. r=edwin
2016-07-16 00:36:56 -07:00
Seth Fowler
8e2ea1af00
Bug 1286165 (Part 3) - Add StreamingLexer tests for the SourceBufferIterator version of Lex(). r=njn
2016-07-15 17:48:57 -07:00
Seth Fowler
4c2a4667a7
Bug 1286165 (Part 2) - Advance decoders' SourceBufferIterator directly in StreamingLexer. r=edwin,njn
2016-07-15 17:48:57 -07:00
Seth Fowler
d1bd597704
Bug 1286165 (Part 1) - Allow advancing a SourceBufferIterator without an IResumable. r=edwin
2016-07-15 17:48:57 -07:00
Seth Fowler
da8a36ae22
Bug 1286170 - Handle move-assignment of SourceBufferIterators correctly. r=edwin
2016-07-15 17:48:57 -07:00
Seth Fowler
a92a8b9e06
Bug 1286168 - Add a RAII class to initialize ImageLib services in GTests. r=njn
2016-07-15 17:48:57 -07:00
Seth Fowler
4be29e46b3
Bug 1285867 (Part 7) - Clean up remaining references to decoder 'data errors' and refer to them as just 'errors'. r=edwin
2016-07-15 16:41:11 -07:00
Seth Fowler
905634be2b
Bug 1285867 (Part 6) - Record Decoder telemetry outside of the loop. r=edwin
2016-07-15 16:41:11 -07:00