Commit Graph

431 Commits

Author SHA1 Message Date
Joe Drew
da2ef18bab Bug 802390 - Gracefully handle shutting down a decoder that hasn't had a chance to do any work. r=jrmuizel 2012-11-16 11:43:24 -08:00
Robert O'Callahan
01b72b4b81 Bug 815593. Don't layerize images that exceed MaxTextureSize. r=mattwoodrow
--HG--
extra : rebase_source : f94300b24b90d063df095324663c9d8fa49e42e9
2012-11-28 15:34:45 +13:00
Josh Matthews
2023afbbb4 Bug 801366 - Restrict GIF pixels to the range of the colormap. r=jrmuizel 2012-11-28 10:41:31 -05:00
Jonathan Kew
abddb4e8e4 bug 815512 pt 1 - on retina-display Macs, moz-icon gets a bitmap that is 2x the 'expected' size. r=joe 2012-11-28 09:04:14 +00:00
Nicholas Nethercote
65cbbd034d Bug 711901 - When possible, measure, don't compute, the size of imgFrame::mImageSurface. r=joedrew.
--HG--
extra : rebase_source : 72dc61c2618166262ec6d83778d7bd2a3a7b5d61
2012-11-26 16:29:56 -08:00
Ehsan Akhgari
aa99449330 Test case for bug 815359 - r=joe 2012-11-27 21:55:10 -05:00
Ehsan Akhgari
6206cca436 Bug 815359 - Use the correct contract ID to create nsISupportsPRUint32's in the icon decoder; r=bzbarsky 2012-11-27 21:07:01 -05:00
Seth Fowler
cf0fa3d371 Bug 813800 - Never revalidate data URIs. r=joe 2012-11-20 16:58:32 -08:00
Ehsan Akhgari
de15f90350 Bug 813787 - Remove support for image-sniffing-services; r=joe 2012-11-20 19:08:06 -05:00
Joe Drew
cc5c8dd282 Bug 812602 - Don't use the DecodeStyle enum namespace, since it doesn't actually exist. r=jrmuizel
--HG--
extra : rebase_source : ac71cc01359c39e275e8b12c44e7d4e5fb04a81b
2012-11-20 14:27:39 -05:00
Tobias Netzel
f754fd5fcb Bug 812932 - Have EndianMacros.h recognize __BIG_ENDIAN__ preprocessor macro - r=bbondy 2012-11-19 09:09:17 -08:00
Jeff Muizelaar
2c4d59c7eb Bug 812602. Don't decode jpegs progressively when we have all the data. r=joe
Decoding progressively is slower because we duplicate work like color
conversion and color correction for each outputted scan. We already suppress
paints when have all of the data, so we shouldn't do this extra work.

--HG--
extra : rebase_source : e6dd733546e1bb9e6e3279d9ceea0b4c0a88e049
2012-11-18 20:18:52 -05:00
Josh Matthews
440d2df25e Bug 803125 - Notify status tracker when image request is destroyed. r=joe 2012-11-17 13:33:20 +00:00
Ryan VanderMeulen
6c954363f7 Backed out changeset fb3e2fb82000 (bug 802390) for mochitest bc orange. 2012-11-16 16:58:11 -05:00
Joe Drew
9775b3728a Bug 802390 - Gracefully handle shutting down a decoder that hasn't had a chance to do any work. r=jrmuizel 2012-11-16 11:43:24 -08:00
Jeff Muizelaar
2535ac7838 Bug 795648. Add a commment describing why we need to call RequestDecode. r=mwoodrow
--HG--
extra : rebase_source : bd0b47c361eee7c0ed6f00f92eda8aabc5ef08e9
2012-11-16 11:00:06 -08:00
Jeff Muizelaar
102fccacd8 Bug 794957. Use imgLoader instead of imgILoader. r=joe
This makes use of imgLoader instead of imgILoader in more places.

--HG--
extra : rebase_source : 6b391e5ecc2fd9558516c7493116e7e7852b71b3
2012-11-16 10:24:58 -08:00
Matt Woodrow
5ca8550984 Bug 795648 - Don't synchronously decode images during painting. r=joe 2012-11-16 10:04:29 -08:00
Honza Bambas
972e6c97df Bug 734062 - Followup: fixed unused varialble use, r=joedrew 2012-11-14 13:56:56 +01:00
Josh Matthews
3c46b167b3 Bug 803121 - Avoid using virtual functions in imgRequestProxy's destructor. r=joe 2012-10-19 15:39:38 -04:00
Jeff Muizelaar
0b4904f43a Bug 811461. Switch IMAGE_DECODE_LATENCY to microseconds. r=taras
Currently this is recording in ms when we expect all of the values to be <5ms
I believe this was just a typo.

--HG--
extra : rebase_source : ed71dc9bd7dc9ef0903e2e84152b44ba13d97adf
2012-11-13 15:38:54 -08:00
Seth Fowler
292e3a2a66 Bug 809181 - SVGDocumentWrapper::UpdateViewportBounds() should check if anything has changed instead of relying on callers. r=dholbert 2012-11-08 14:54:47 -05:00
Honza Bambas
08beb17936 Bug 734062 - stale image in the image cache when an updated version is fetched into HTML5 offline cache in background, r=joedrew 2012-11-08 19:54:00 +01:00
Josh Matthews
789d116d2c Bug 805013 - Paper over the browser-chrome crashes from certain circumstances that run ~RasterImage off the main thread. r=joe 2012-11-07 17:25:27 -05:00
Josh Matthews
9efa1b8f4d Bug 805013 - Diagnostic bandaid to ensure that all RasterImages are removed from the discard tracker. r=joe 2012-11-07 14:03:10 -05:00
Jeff Walden
24e7854e4d Bug 803439 - Add removeFrom() that takes a list and asserts the element is initially present in it. r=jlebar
--HG--
extra : rebase_source : c2aad35774a71167f97f940fb416735862b78950
2012-10-23 17:43:23 -07:00
aceman
5d56c2ede7 Bug 808379 - fix Assertion failure: isEmpty(), at ../../dist/include/mozilla/LinkedList.h:258 causing Thunderbird crash since landing of fix to bug 803688. r=jlebar
Clear the sDiscardableImages linked list in DiscardTracker::Shutdown() so that the list destructor finds an empty array, which is required after bug 803688.
2012-11-04 23:20:52 +00:00
Seth Fowler
7864e2fd96 Bug 807556 - Remove imgIRequest::loadImage's aRequest argument. r=joe 2012-11-03 23:04:07 -04:00
Kyle Huey
0469a02b25 Bug 798491: Add an option to stick all chrome JSMs/JS components in the same compartment. r=mrbkap,philikon
--HG--
extra : rebase_source : 98337b6a8c07d05e8c961a452dd05a7d75c3c60b
2012-10-31 09:13:28 -07:00
Vicamo Yang
a78b5b38c7 Backout 67cb43bb8865: Breaks B2G Marionette 2012-10-31 16:35:11 +08:00
Kyle Huey
9a3674d80a Bug 798491: Add an option to stick all chrome JSMs/JS components in the same compartment. r=mrbkap,philikon 2012-10-30 12:28:11 -07:00
Ed Morley
4144129728 Backout a145ded68994, e0cf397089ec & 1545e91c658e (bug 798491) for bustage on a CLOSED TREE 2012-10-30 17:02:31 +00:00
Kyle Huey
50b5d3f681 Bug 798491: Add an option to stick all chrome JSMs/JS components in the same compartment. r=mrbkap,philikon 2012-10-30 08:26:12 -07:00
Nathan Froyd
4c61ef9ec5 Bug 806618 - rewrite PR_NewLogModule calls to not generate static initializers; r=ehsan 2012-10-29 19:32:10 -04:00
Jacek Szpot
761b11160f Bug 792180 - Replace NS_{UN,}LIKELY with MOZ_{UN,}LIKELY; r=ehsan 2012-10-26 15:32:10 +02:00
Jeff Muizelaar
b16e964639 Bug 805119. Add a note about the deprecation of findEntryProperties. r=joe
DONTBUILD
2012-10-24 14:40:41 -04:00
Josh Matthews
6308bec76a Bug 802485 - Quiet intermittent assertion for further investigation. r=orange 2012-10-22 15:40:05 -04:00
Nick Hurley
6e789d3d04 Bug 536324 part 2 - Change implementers of nsIChannel. r=sworkman 2012-10-22 10:51:07 -07:00
Abhishek Potnis
ad50b9e765 Bug 795511: Removed usages of PR_BIT and PR_BITMASK from the tree; r=ehsan 2012-10-20 07:31:43 +05:30
Josh Matthews
ea1bd9cec2 Bug 802485 - Remove special handling for canceled image request proxies that are changing owners. r=joe 2012-10-22 12:44:53 -04:00
Joe Drew
9d1e0f23ac Bug 802168 - Don't shut down the decoder when we're destroying RasterImages; instead, just remove ourselves from the decoding list. r=jrmuizel,jdm 2012-10-22 13:16:57 -04:00
Phil Ringnalda
70317b582e Bug 761049 - Back out 6ba8fbe719cc to reenable browser_bug666317.js on Linux 2012-10-19 21:59:18 -07:00
Seth Fowler
0a022d333f Bug 803349 - Restore NS_DECL_IMGICONTAINER to VectorImage / RasterImage. r=joe 2012-10-19 13:27:11 -07:00
Isaac Aggrey
c984edf823 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
2409f6ad32 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
84d6a72931 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
067486383c Bug 801358 - Part 1 - Discard downscaled frames. r=jrmuizel
--HG--
extra : rebase_source : 1b699918323e7c116521cbde579aafb07846b2f6
2012-10-13 22:17:30 -04:00
Joe Drew
21be03f840 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
4baaf41686 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
1f866d258a 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
dace21eaac 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
7a3f9166ff 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
c4ae9ab2b8 Bug 802409: Mark a bunch of /image subdirectories as FAIL_ON_WARNINGS. r=joe 2012-10-17 08:32:12 -07:00
Daniel Holbert
c7e369ef96 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
348f30fb12 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
00c4850b48 Bug 801405 - Avoid attempting to optimize frames that no longer exist. r=joe 2012-10-15 18:06:43 -04:00
Josh Matthews
34c635b69a 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
743675454b 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
16a5b894ba bug 801013 - imgRequest::AdjustPriority wrongly adjusts loadgroup instead of channel r=joedrew 2012-10-13 12:47:36 -04:00
Jeff Muizelaar
8b8ae578b1 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
c8eb573b6f 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
2eae9dea92 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
c15377e449 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
dd562843b4 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
fdb41f04a2 Bug 505385 - Part 14: Update image request status on decoding errors. r=joe 2012-10-12 12:11:22 -04:00
Josh Matthews
f905bcf9ea 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
c41e3878e6 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
b84562cef3 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
8d5ea85e2e 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
f6be2abef0 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
8574293a68 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
9bf020119e 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
e200b97e86 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
67a6f4a027 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
5150ce1dfa Bug 505385 - Part 4: Fix some warnings. r=joe 2012-10-12 12:11:20 -04:00
Bobby Holley
3d1f176ae5 Bug 505385 - Part 2: Move notification dispatch into imgStatusTracker. r=joe 2012-10-12 12:11:20 -04:00
Bobby Holley
efe6cefdbd Bug 505385 - Part 1: Hoist proxy list into imgStatusTracker. r=joe 2012-10-12 12:11:20 -04:00
Bobby Holley
a30c1baa07 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
138935b14a Back out bug 585385 for build bustage. 2012-10-11 21:58:24 -04:00
Josh Matthews
fcd1ce5c84 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
32a0b22fe3 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
2bdbf552ac 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
ea0e69f95d 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
a03358d4e3 Bug 505385 - Part 14: Update image request status on decoding errors. r=joe 2012-10-11 21:34:24 -04:00
Josh Matthews
1d3700a537 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
1013622191 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
abbb0715c2 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
5690e9485e 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
228592e98b 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
8634a5ed5c 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
de02cdd129 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
e1ff93956a 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
faa13681c0 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
Bobby Holley
293f4dd9eb Bug 505385 - Part 4: Fix some warnings. r=joe 2012-10-11 21:34:22 -04:00
Bobby Holley
147dfd5698 Bug 505385 - Part 2: Move notification dispatch into imgStatusTracker. r=joe 2012-10-11 21:34:22 -04:00
Bobby Holley
9248a7c43a Bug 505385 - Part 1: Hoist proxy list into imgStatusTracker. r=joe 2012-10-11 21:34:22 -04:00
Bobby Holley
d0c2f1c51f 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-11 21:34:22 -04:00
Trevor Saunders
fa29be8178 bug 797445 - remove prmem.h from imagelib r=joe 2012-10-03 14:10:31 -04:00
Kyle Huey
27f7c5c4e0 Bug 799791: Remove unused boolean parameters and dead branches. r=jlebar 2012-10-11 09:35:43 -07:00
Kyle Huey
2d6c2a4bf7 Bug 789846: Ensure that we unblock unload if we're only pretending that the request finished. r=joe 2012-10-09 09:47:14 -07:00