Commit Graph

338 Commits

Author SHA1 Message Date
Isaac Aggrey
9e41ce2fa0 Bug 794510: Part 2: Use more stdint types instead of protypes.h types; r=ehsan 2012-10-11 18:38:04 -05:00
Nicholas Nethercote
b7c50cf8b3 Bug 744686 - Comment out unused |alpha_bits| variable to fix a compile warning. r=jmuizelaar.
--HG--
extra : rebase_source : 48be31fe3eab889e6f7930f44b96140ba076c389
2012-10-17 15:54:25 -07:00
Joe Drew
e339553286 Bug 801358 - Part 2 - Account for downscaled frames in about:memory. r=jrmuizel
--HG--
extra : rebase_source : c5a507f0f5d4fa5b700f19cdb359fd8281baa80d
2012-10-13 22:17:40 -04:00
Joe Drew
04467ccef4 Bug 801358 - Part 1 - Discard downscaled frames. r=jrmuizel
--HG--
extra : rebase_source : 1b699918323e7c116521cbde579aafb07846b2f6
2012-10-13 22:17:30 -04:00
Joe Drew
5dcf25c35e Bug 795940 - Part 0.5 - Move ScaleRequest and the runners into the CPP file so consumers don't have to care about them. BY THE DEMAND OF AND r=jrmuizel
--HG--
extra : rebase_source : eeb96eb9680017d2be9ee99ac0c760249a74bcbe
2012-10-12 18:24:47 -04:00
Joe Drew
3f8da7e7b0 Bug 795940 - Part 0.4 - Only try to downscale images that have a single instance on a web page, since that's all we support anyhow. r=jrmuizel
--HG--
extra : rebase_source : 1bda15c8edb7e2d1d3ca26354ce8429640b03b43
2012-10-12 18:20:25 -04:00
Joe Drew
259989f4be Bug 795940 - Part 0.3 - Support stopping requests (on a best-effort basis). r=jrmuizel
--HG--
extra : rebase_source : 793d04fa7954bbf060f01436bc0d257edd05a28a
2012-10-12 18:20:22 -04:00
Joe Drew
b6f2e527c1 Bug 795940 - Part 0.2 - Change from singleton ScaleWorker and DrawWorker (and on-demand allocated ScaleRequest) to nsIRunnables, allocated on demand, that know how to run themselves. r=jrmuizel
--HG--
extra : rebase_source : 9b98ed62eaf5fdddf6ee117ec5f26d049d752eaf
2012-10-12 18:20:19 -04:00
Joe Drew
fa19aa815b Bug 795940 - Part 0.1 - Allocate ScaleRequests dynamically and only hold on to a ScaleResult in RasterImage. Let ScaleRequests only hold a weak pointer to RasterImage so their lifetimes aren't bound. Finally, make ScaleRequests hold on to references to their surfaces instead of imgFrames. r=jlebar,jrmuizel
--HG--
extra : rebase_source : 335837e469e8318f0a70907060470b0a0e14ef21
2012-10-10 11:35:23 -04:00
Daniel Holbert
8f23c980e4 Bug 802409: Mark a bunch of /image subdirectories as FAIL_ON_WARNINGS. r=joe 2012-10-17 08:32:12 -07:00
Daniel Holbert
7d06b363a8 Bug 802458: Remove useless "NS_SUCCEEDED(rv)" in icon/gtk/nsIconChannel.cpp, and declare rv down where it's used. r=karlt 2012-10-16 22:18:10 -07:00
Josh Matthews
253a471ccd Bug 802144 - Don't delete memory used for image frames while the decoder could still try to use it. r=joe 2012-10-16 22:45:14 -04:00
Josh Matthews
128fc4e480 Bug 801405 - Avoid attempting to optimize frames that no longer exist. r=joe 2012-10-15 18:06:43 -04:00
Josh Matthews
c9846aa0ec Bug 801701 - Fix references to non-existent requests in status trackers for extracted frames. r=joe 2012-10-15 16:05:01 -04:00
Ehsan Akhgari
b3077d258d Silence a compiler warning about initialization order of RasterImage members, no bug
--HG--
extra : rebase_source : a109b2d20f362add2dbb38a6b2037adfab69941e
2012-10-13 17:28:39 -04:00
Patrick McManus
8b408af6a4 bug 801013 - imgRequest::AdjustPriority wrongly adjusts loadgroup instead of channel r=joedrew 2012-10-13 12:47:36 -04:00
Jeff Muizelaar
79fa5e64ff Bug 799335. Fix unintenional image flash on tab switch. r=joe,dholbert
This was caused by bug 792199.

There are two parts to this fix:
1. Always decode during StartDecoding even if we already have a Decoder
(hopefully this is safe) This was a bug in the original patch.

2. After calling StartDecoding() we recheck if we're complete and abort if
we're not doing a SYNC_DECODE. Before this regression we would usually be done
decoding the image because of the decoding we did when we Locked all of the
images on tab switch.

--HG--
extra : rebase_source : 9db93075aad2f45c6cc70d27ef67468b4507642c
2012-10-12 16:04:32 -04:00
Josh Matthews
e5b76c73ef Bug 505385 - Part 18: Reduce number of notifications - START_REQUEST/START_CONTAINER -> SIZE_AVAILABLE, remove START_DECODE and START_FRAME. Rename remaining ones for clarity. r=joe 2012-10-12 12:11:23 -04:00
Josh Matthews
dd334e9c67 Bug 505385 - Part 17: Clean up unused arguments in imgStatusTracker observers, document unclear things, etc. r=joe 2012-10-12 12:11:23 -04:00
Josh Matthews
0783fb9f7b Bug 505385 - Part 17: Consolidate image request failure handling code into a central location and deal with preexisting errors correctly. r=joe 2012-10-12 12:11:23 -04:00
Josh Matthews
7a220dd288 Bug 505385 - Part 15: Fix cases where pending requests in nsImageLoadingContent could be cancelled and never receive OnStartRequest. r=joe 2012-10-12 12:11:22 -04:00
Josh Matthews
13ed44dbb4 Bug 505385 - Part 14: Update image request status on decoding errors. r=joe 2012-10-12 12:11:22 -04:00
Josh Matthews
e9be8ac74c Bug 505385 - Part 13: Remove OnStopContainer and make OnStopDecode a true decode notification. r=joe 2012-10-12 12:11:22 -04:00
Josh Matthews
1e2e233108 Bug 505385 - Part 12: Ensure clones of static image proxies result in more static proxies. r=joe 2012-10-12 12:11:22 -04:00
Josh Matthews
31e9ef66cd Bug 505385 - Part 11: Create a new imgINotificationObserver interface to replace all uses of imgIContainerObserver and imgIDecoderObserver outside of image/. r=joe 2012-10-12 12:11:22 -04:00
Josh Matthews
c34861bb2d Bug 505385 - Part 10: Remove GetConsumers from imgStatusTracker by delegating all remaining notification operations from imgRequest to the status tracker. r=joe 2012-10-12 12:11:21 -04:00
Josh Matthews
4b1668f759 Bug 505385 - Part 9: Hoist decoder and container observer out of imgRequest into imgStatusTracker. r=joe 2012-10-12 12:11:21 -04:00
Josh Matthews
6348c50b96 Bug 505385 - Part 8: Remove image reference from proxies, and delegate operations on images to the owning request or the status tracker. r=joe 2012-10-12 12:11:21 -04:00
Bobby Holley
b215db7b80 Bug 505385 - Part 7: Initialize imgRequestProxy with a status tracker. r=joe
This is sort of symbolic for now, since we keep mOwner and mImage and just
pull them off the tracker, but we'll soon change that. ;-)
2012-10-12 12:11:21 -04:00
Bobby Holley
ad6877697a Bug 505385 - Part 6: Expose imgRequest::GetStatusTracker, and ensure that it's never null. r=joe
We're slowly turning imgStatusTracker into the central coordination point between imgRequest,
imgRequestProxy, and Image, so we need it to be a bit more visible.
2012-10-12 12:11:21 -04:00
Bobby Holley
26f4d62fed Bug 505385 - Part 5: Expose pointers to the imgRequest and Image on imgStatusTracker. r=joe
This will help us decouple imgRequest and imgRequestProxy.
2012-10-12 12:11:20 -04:00
Bobby Holley
c4499630a2 Bug 505385 - Part 4: Fix some warnings. r=joe 2012-10-12 12:11:20 -04:00
Bobby Holley
4575bcf375 Bug 505385 - Part 2: Move notification dispatch into imgStatusTracker. r=joe 2012-10-12 12:11:20 -04:00
Bobby Holley
e011cb0dc6 Bug 505385 - Part 1: Hoist proxy list into imgStatusTracker. r=joe 2012-10-12 12:11:20 -04:00
Bobby Holley
bd4348cc2d Bug 505385 - Part 0: Remove the last vestiges of manual animation reset within imagelib. r=joe
This stuff should now all be taken care of by the AnimationConsumers infrastructure.
2012-10-12 12:11:20 -04:00
Josh Matthews
e82f309179 Back out bug 585385 for build bustage. 2012-10-11 21:58:24 -04:00
Josh Matthews
d64a2c9b5a Bug 505385 - Part 18: Reduce number of notifications - START_REQUEST/START_CONTAINER -> SIZE_AVAILABLE, remove START_DECODE and START_FRAME. Rename remaining ones for clarity. r=joe 2012-10-11 21:34:24 -04:00
Josh Matthews
e93d04a3ed Bug 505385 - Part 17: Clean up unused arguments in imgStatusTracker observers, document unclear things, etc. r=joe 2012-10-11 21:34:24 -04:00
Josh Matthews
09e583e03a Bug 505385 - Part 17: Consolidate image request failure handling code into a central location and deal with preexisting errors correctly. r=joe 2012-10-11 21:34:24 -04:00
Josh Matthews
80d0bbfb1b Bug 505385 - Part 15: Fix cases where pending requests in nsImageLoadingContent could be cancelled and never receive OnStartRequest. r=joe 2012-10-11 21:34:24 -04:00
Josh Matthews
41e3c3120f Bug 505385 - Part 14: Update image request status on decoding errors. r=joe 2012-10-11 21:34:24 -04:00
Josh Matthews
db4200e616 Bug 505385 - Part 13: Remove OnStopContainer and make OnStopDecode a true decode notification. r=joe 2012-10-11 21:34:23 -04:00
Josh Matthews
4fec30568a Bug 505385 - Part 12: Ensure clones of static image proxies result in more static proxies. r=joe 2012-10-11 21:34:23 -04:00
Josh Matthews
d09d8af2a1 Bug 505385 - Part 11: Create a new imgINotificationObserver interface to replace all uses of imgIContainerObserver and imgIDecoderObserver outside of image/. r=joe 2012-10-11 21:34:23 -04:00
Josh Matthews
d6ba50447f Bug 505385 - Part 10: Remove GetConsumers from imgStatusTracker by delegating all remaining notification operations from imgRequest to the status tracker. r=joe 2012-10-11 21:34:23 -04:00
Josh Matthews
679f2d5e91 Bug 505385 - Part 9: Hoist decoder and container observer out of imgRequest into imgStatusTracker. r=joe 2012-10-11 21:34:23 -04:00
Josh Matthews
dd877db859 Bug 505385 - Part 8: Remove image reference from proxies, and delegate operations on images to the owning request or the status tracker. r=joe 2012-10-11 21:34:23 -04:00
Bobby Holley
76c30997de Bug 505385 - Part 7: Initialize imgRequestProxy with a status tracker. r=joe
This is sort of symbolic for now, since we keep mOwner and mImage and just
pull them off the tracker, but we'll soon change that. ;-)
2012-10-11 21:34:22 -04:00
Bobby Holley
89f6f2488d Bug 505385 - Part 6: Expose imgRequest::GetStatusTracker, and ensure that it's never null. r=joe
We're slowly turning imgStatusTracker into the central coordination point between imgRequest,
imgRequestProxy, and Image, so we need it to be a bit more visible.
2012-10-11 21:34:22 -04:00
Bobby Holley
3f63b1f753 Bug 505385 - Part 5: Expose pointers to the imgRequest and Image on imgStatusTracker. r=joe
This will help us decouple imgRequest and imgRequestProxy.
2012-10-11 21:34:22 -04:00