Commit Graph

592 Commits

Author SHA1 Message Date
Timothy Nikkel
76cd084078 Bug 784591. Just end the crashtest ownerdiscard.html after 30 seconds if the image doesn't get decoded. r=jdm
The test normally ends when the image is discarded. But there is nothing in the test that requires the image to ever be decoded. To properly test the original bug that this test is intended for we want the following sequence: imgRequestProxy::Cancel, imgRequestProxy::ChangeOwner, and then the image getting discarded. The cancel call makes it hard to get a full decode started, especially since it needs to happen before the ChangeOwner call which itself happens quite early. So it appears that there is no easy way to modify the test to keep this sequence of events 100% of the time.

However the test as it currently stands does indeed get this sequence of events most of the time, so it continues to test the right thing most of the time. The few times it doesn't test the right thing we can just move on.
2013-02-13 14:18:07 -06:00
David Zbarsky
b6cdb97b40 Bug 833446: Remove nsIDOMSVGSVGElement r=peterv 2013-02-08 14:55:07 -05:00
Ed Morley
ae0aa82de4 Backout changeset d2fda338874d (bug 784591) for causing bug 832571 again 2013-02-07 11:20:47 +00:00
Timothy Nikkel
5dae076200 Bug 784591. Wait for the DOMContentLoaded event in the child iframe before removing the node from the document in this test. r=jdm 2013-02-06 19:24:15 -06:00
Kyle Huey
f3832bd24f Bug 835814. r=joe 2013-02-05 09:38:22 +00:00
Seth Fowler
e546fdba5b Bug 836155 - Replace GetCurrentFrameRect() with FrameRect(aWhichFrame). r=joe 2013-02-04 16:06:14 -08:00
Seth Fowler
e315e0e91d Bug 836124 - Replace GetCurrentFrameIsOpaque() with [noscript] FrameIsOpaque(aWhichFrame). r=joe 2013-02-04 14:22:30 -08:00
Daniel Holbert
c5e34e20a7 Bug 837313: Mark image/decoders, image/decoders/icon, and image/decoders/icon/win as FAIL_ON_WARNINGS. r=joe 2013-02-01 23:19:15 -08:00
Seth Fowler
db1690f6db Bug 828176 - Make RasterImage::GetURIString work again. r=joe 2013-01-30 12:11:20 -08:00
Ryan VanderMeulen
949324d59b Backed out changeset 25ea9ac37380 (bug 828176) for crashes. 2013-01-30 18:16:09 -05:00
Seth Fowler
e8e248dbc4 Bug 828176 - Make RasterImage::GetURIString work again. r=joe 2013-01-10 18:15:05 -08:00
Chris Peterson
142dba2bef Bug 834611 - Part 3: Mark widget/android, mozglue/android, and image/decoders/icon/android as FAIL_ON_WARNINGS. r=kats 2013-01-25 11:03:56 -08:00
Seth Fowler
00c31ffd62 Bug 820602 - Remove change to decoder notification behavior that causes AWSY leak. r=joe 2013-01-25 18:39:11 -08:00
Joe Drew
441cdfe6be Bug 832446 - Don't use the high-quality downscaler on multipart images. r=jrmuizel 2013-01-21 11:37:54 -05:00
Ehsan Akhgari
893e6d1c91 Follow-up for bug 833097, bug 833101 and bug 833118 - Only disable PGO on the modules in question on Windows, a=me
--HG--
extra : rebase_source : dae7ac9db9e0b85a39cf7d647b46f9da48398dc2
2013-01-21 18:53:05 -05:00
Christian Holler
63a38db7f4 Bug 797900 - Disable invalid-size-second-frame crashtest under AddressSanitizer. r=dbaron, a=nonlibxul 2013-01-22 00:46:40 +01:00
Joe Drew
8a609d3625 Bug 833101 - Stop doing PGO on imagelib in the hopes of being able to reopen the tree; r=ehsan a=dePGO 2013-01-21 17:18:05 -05:00
Mats Palmgren
d5826be6d7 Bug 786533 - Replace NS_MIN/NS_MAX with std::min/std::max and #include <algorithm> where needed. r=ehsan 2013-01-15 13:22:03 +01:00
Kyle Huey
66c651ccc9 Bug 824658 - Avoid infinite decode loops of images by switching some GetFrame calls to GetFrameNoDecode. r=joe 2013-01-14 16:01:08 -05:00
Ryan VanderMeulen
e59d7421d3 Backed out changeset c8fe09ac1842 (bug 825720) on suspicion of making bug 772443 nearly perma-orange. 2013-01-12 21:00:30 -05:00
Seth Fowler
2a183d535a Bug 825720 - Don't use GetRootLayoutFrame to calculate the size of SVG images. r=joe 2013-01-10 19:38:34 -08:00
Chris Peterson
b483c0b61c Bug 829344 - Part 4: Fix gcc -Wlogical-op warnings from PRBool/bool conversion. r=ehsan 2013-01-10 17:56:51 -08:00
Filippo Cristofoletti
55990a5578 Bug 243763 - explicit cast from LONG to int8_t. r=bbondy 2013-01-11 12:05:31 -05:00
Andrew Halberstadt
3a19a47088 Bug 811779 - Enable larger set of reftests on B2G emulators, r=jgriffin 2013-01-03 15:35:01 -05:00
David Zbarsky
f95b0f0bde Bug 827119: Move SVGSVGElement to mozilla::dom r=bz
--HG--
rename : content/svg/content/src/nsSVGSVGElement.cpp => content/svg/content/src/SVGSVGElement.cpp
rename : content/svg/content/src/nsSVGSVGElement.h => content/svg/content/src/SVGSVGElement.h
2013-01-09 18:02:45 -05:00
Seth Fowler
bf6d29e9b6 Bug 822846 - Use nsMimeTypes.h instead of working with literal strings in imagelib. r=joe 2013-01-08 13:40:47 -08:00
Jonathan Watt
7ab26ac57a Add a comment to SVGDocumentWrapper::OnStopRequest. DONTBUILD (NPOB). No bug. 2012-12-27 17:29:03 +00:00
Masatoshi Kimura
285c17a7c5 Bug 826613 - Enable FAIL_ON_WARNINGS on MSVC in image/. r=joe 2013-01-05 12:55:23 +09:00
Justin Lebar
f3f2216930 Bug 825519 - Add telemetry for the max number of times any one image was decoded. r=joe 2013-01-03 16:06:08 -05:00
Ehsan Akhgari
47f5dc78a2 Backed out changeset 3d51d7f54e5d (bug 825519) because of build failures 2013-01-03 13:28:18 -05:00
Justin Lebar
2cbdb7986c Bug 825519 - Add telemetry for the max number of times any one image was decoded. r=joe 2013-01-03 12:52:58 -05:00
Ehsan Akhgari
915062dfb0 Bug 825527 - Part 1: Look through an Xray wrapper at our images in tests so that we always see the QueryInterface method on them; r=bzbarsky 2013-01-02 21:26:43 -05:00
Masatoshi Kimura
ecc812a3c0 Bug 824247 - Part 3: Disable FAIL_ON_WARNINGS on not-yet-warning-free directories on MSVC. r=ted 2013-01-03 00:56:38 +09:00
Nicholas Nethercote
b6939f2c42 Bug 819819 (part 2) - Remove |name| from NS_MEMORY_REPORTER_MALLOC_SIZEOF_FUN. r=jlebar.
--HG--
rename : dom/tests/mochitest/localstorage/test_localStorageQuotaPrivateBrowsing_perwindowpb.html => dom/tests/mochitest/localstorage/test_localStorageQuotaPrivateBrowsing.html
rename : toolkit/components/places/tests/browser/browser_favicon_privatebrowsing_perwindowpb.js => toolkit/components/places/tests/browser/browser_favicon_privatebrowsing.js
extra : rebase_source : 557cc60522970ce7eea3b006d357284d9b4694c4
2012-12-23 16:59:51 -08:00
Ryan VanderMeulen
d0c3b92170 Backed out changeset 5244caae8c46 (bug 822846) for Windows build bustage. 2012-12-23 11:45:38 -05:00
Seth Fowler
d8f9d47626 Bug 822846 - Use nsMimeTypes.h instead of working with literal strings in imagelib. r=joe 2012-12-19 13:33:56 -08:00
Seth Fowler
35e2d6a5da Bug 822526 - Turn mozilla::image::Image into an interface. r=joe 2012-12-19 14:24:32 -08:00
Seth Fowler
3b8e9ef295 Bug 821023 - imgStatusTracker should not know about imgRequest. r=joe 2012-12-19 13:28:54 -08:00
Neil Rashbrook
d8fe604ecd Bug 818861 Shouldn't allow promising a flat string from a flat string r=dbaron 2012-12-22 20:40:37 +00:00
Nicholas Nethercote
081c9df69e Bug 820132 - Measure imgFrame::mOptSurface's size where possible, instead of calculating it. r=joedrew.
--HG--
extra : rebase_source : da7616356429242e924a09df1124b1dbab13ad60
2012-12-18 21:59:30 -08:00
Nicholas Nethercote
23a8027808 Bug 820128 - Use moz_malloc_size_of to measure "images-content-used-uncompressed". r=jlebar.
--HG--
extra : rebase_source : 07e397f24a74b0e534d8ebbab53a48b7cbc3febc
2012-12-18 14:50:59 -08:00
Joe Drew
8f36201dd5 Bug 716140 - Convert the XPCOM imgIDecoderObserver interface into a private C++ interface, imgDecoderObserver, so we can more easily make changes to it. r=jrmuizel 2012-12-18 11:37:15 -05:00
Joe Drew
ea276a720a Bug 716140 - Merge imgIContainerObserver and imgIDecoderObserver since they aren't used in different places any more. r=jrmuizel 2012-12-18 11:37:15 -05:00
Joe Drew
c87f937e12 Bug 716140 - Don't use Gets on mImage from decoders. 2012-12-18 11:37:14 -05:00
Joe Drew
16cb6b38fa Bug 716140 - Move imgStatusTrackerObserver to the cpp file. r=jrmuizel 2012-12-18 11:37:14 -05:00
Seth Fowler
581f4c8fce Bug 816374 - Don't export RasterImage from imagelib. r=joe 2012-12-17 18:35:46 -08:00
Seth Fowler
60f17bd025 Bug 816362 - Refactor imgTools::DecodeImageData to remove imgIContainer argument and use ImageFactory. r=joe 2012-12-17 17:35:07 -08:00
Seth Fowler
8ab5195c63 Bug 815471 - Don't assume that all imgIContainers are either a RasterImage or a VectorImage. r=joe 2012-12-17 14:05:18 -08:00
Ryan VanderMeulen
41d18e74d0 Backed out 4 changesets (bug 815471, bug 821023, bug 816374, bug 816362) for reftest failures. 2012-12-16 21:13:35 -05:00
Seth Fowler
2b3b199964 Bug 821023 - imgStatusTracker should not know about imgRequest. r=joe 2012-12-14 15:42:18 -08:00
Seth Fowler
9b81aa5303 Bug 816374 - Don't export RasterImage from imagelib. r=joe 2012-12-03 16:22:32 -08:00
Seth Fowler
2401948cc4 Bug 816362 - Refactor imgTools::DecodeImageData to remove imgIContainer argument and use ImageFactory. r=joe 2012-12-13 17:02:26 -08:00
Seth Fowler
941299b5d8 Bug 815471 - Don't assume that all imgIContainers are either a RasterImage or a VectorImage. r=joe 2012-12-13 16:06:31 -08:00
Jeff Walden
e0523203c9 Bug 820570 - Move mozilla::DebugOnly into DebugOnly.h to pare down the grab-baggish Util.h. r=Ms2ger 2012-12-14 18:58:45 -05:00
Jeff Muizelaar
87d9a43558 Bug 811891. Only sync decode once per image. r=joe
Bug 799335 intentionally caused us to do some decoding everytime StartDecoding
is called. This can be multiple times per paint which can delay tab switch
substantially.

--HG--
extra : rebase_source : d3f151e85c142984c39f21ff7e13d1d37d39a4bf
2012-12-15 14:03:42 -05:00
Jeff Muizelaar
5a168d2ab1 Bug 801061. Switch from imgIRequest to imgRequestProxy. r=joe
This doesn't switch all of the users yet, but is a step in the right
direction.

--HG--
extra : rebase_source : 91b4fef3f67586179c119208d000cf7629e04963
2012-10-12 08:43:01 -04:00
Jeff Muizelaar
10f8bed215 Bug 821448. Make a bunch of imagelib classes builtinclass. r=joe
We don't want script implementing these.

--HG--
extra : rebase_source : dfe4a8c3f201e6dab5bd6c055fa10463a8f00029
2012-12-14 14:42:36 -05:00
Adam Dane [:hobophobe]
fa56bc0fa9 Bug 819412 - Test for Bug 733553 gets desynched under load r=joe 2012-12-10 15:13:42 -06:00
Saurabh Anand
151d13c7a5 Bug 818817 - Fix some compiler warnings, r=Ms2ger 2012-12-09 22:53:19 +05:30
Patrick McManus
d10d748869 bug 792438 - part 0. imagelib proxies should share load group root r=joedrew 2012-12-04 18:06:29 -05:00
Joe Drew
6f8d805500 Bug 804559 - Use correct rounding code instead of rolling our own. r=jrmuizel
Drew's First Law of Programming: If you write your own rounding code, you will
be wrong 100% of the time.
2012-12-03 18:02:11 -05:00
Joe Drew
958d3f216d 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
f9199a2935 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
bc180e53ed Bug 801366 - Restrict GIF pixels to the range of the colormap. r=jrmuizel 2012-11-28 10:41:31 -05:00
Jonathan Kew
baa5823065 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
da226d8e42 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
3b0448a7df Test case for bug 815359 - r=joe 2012-11-27 21:55:10 -05:00
Ehsan Akhgari
6339b343b1 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
e4d770fba5 Bug 813800 - Never revalidate data URIs. r=joe 2012-11-20 16:58:32 -08:00
Ehsan Akhgari
308de53b5b Bug 813787 - Remove support for image-sniffing-services; r=joe 2012-11-20 19:08:06 -05:00
Joe Drew
28e55e16c7 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
dc07531bb5 Bug 812932 - Have EndianMacros.h recognize __BIG_ENDIAN__ preprocessor macro - r=bbondy 2012-11-19 09:09:17 -08:00
Jeff Muizelaar
a30ad6a153 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
703991dff3 Bug 803125 - Notify status tracker when image request is destroyed. r=joe 2012-11-17 13:33:20 +00:00
Ryan VanderMeulen
7ac458881d Backed out changeset fb3e2fb82000 (bug 802390) for mochitest bc orange. 2012-11-16 16:58:11 -05:00
Joe Drew
ef2d115b6c 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
63feebb7e0 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
985473d336 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
52d048cbef Bug 795648 - Don't synchronously decode images during painting. r=joe 2012-11-16 10:04:29 -08:00
Honza Bambas
902d604757 Bug 734062 - Followup: fixed unused varialble use, r=joedrew 2012-11-14 13:56:56 +01:00
Josh Matthews
d7b09b969f Bug 803121 - Avoid using virtual functions in imgRequestProxy's destructor. r=joe 2012-10-19 15:39:38 -04:00
Jeff Muizelaar
be6b1fe272 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
5d7f1cae01 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
33e46a6177 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
543ca63cc0 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
65b46af47f 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
1677629f44 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
88970df015 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
f5d66e49dc Bug 807556 - Remove imgIRequest::loadImage's aRequest argument. r=joe 2012-11-03 23:04:07 -04:00
Kyle Huey
45adfbf2c8 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
e6e6deb486 Backout 67cb43bb8865: Breaks B2G Marionette 2012-10-31 16:35:11 +08:00
Kyle Huey
7457de64ad 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
a7bb0ec116 Backout a145ded68994, e0cf397089ec & 1545e91c658e (bug 798491) for bustage on a CLOSED TREE 2012-10-30 17:02:31 +00:00
Kyle Huey
dd0cfd935a 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
f9b2a74082 Bug 806618 - rewrite PR_NewLogModule calls to not generate static initializers; r=ehsan 2012-10-29 19:32:10 -04:00
Jacek Szpot
bea8356b2c Bug 792180 - Replace NS_{UN,}LIKELY with MOZ_{UN,}LIKELY; r=ehsan 2012-10-26 15:32:10 +02:00
Jeff Muizelaar
096b94532d Bug 805119. Add a note about the deprecation of findEntryProperties. r=joe
DONTBUILD
2012-10-24 14:40:41 -04:00
Josh Matthews
70206b30d9 Bug 802485 - Quiet intermittent assertion for further investigation. r=orange 2012-10-22 15:40:05 -04:00
Nick Hurley
36fa1cbed9 Bug 536324 part 2 - Change implementers of nsIChannel. r=sworkman 2012-10-22 10:51:07 -07:00
Abhishek Potnis
8ad6e4b4f3 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
4411ead4c2 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
e32a2767e5 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
54142e8123 Bug 761049 - Back out 6ba8fbe719cc to reenable browser_bug666317.js on Linux 2012-10-19 21:59:18 -07:00
Seth Fowler
6d1409c12c Bug 803349 - Restore NS_DECL_IMGICONTAINER to VectorImage / RasterImage. r=joe 2012-10-19 13:27:11 -07:00
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
Bobby Holley
2660c14e19 Bug 505385 - Part 4: Fix some warnings. r=joe 2012-10-11 21:34:22 -04:00
Bobby Holley
202b6c0ce8 Bug 505385 - Part 2: Move notification dispatch into imgStatusTracker. r=joe 2012-10-11 21:34:22 -04:00
Bobby Holley
0e79614bc1 Bug 505385 - Part 1: Hoist proxy list into imgStatusTracker. r=joe 2012-10-11 21:34:22 -04:00
Bobby Holley
bb7d1c3abe 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
05978c735c bug 797445 - remove prmem.h from imagelib r=joe 2012-10-03 14:10:31 -04:00
Kyle Huey
923f1139b2 Bug 799791: Remove unused boolean parameters and dead branches. r=jlebar 2012-10-11 09:35:43 -07:00
Kyle Huey
11b8389ee5 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
Adam Dane [:hobophobe]
a7a2243900 Bug 793585 - Only default the shared state if it's not there. r=joe 2012-10-08 16:09:11 -05:00
Neil Rashbrook
0c86b78e7a Bug 794884 Followup to export SupportImageWithMimeType from libxul for mimei.cpp r=joe 2012-10-07 19:01:08 +01:00
Ed Morley
bd0d1f6c0a Bug 795393 - Remove instances of "ifneq (mobile,$(MOZ_BUILD_APP))" since they're now always true; r=ted 2012-10-06 16:46:09 +01:00
Justin Dolske
d4e138c368 Bug 793366 - Add noise to the white background used for transparent images in image documents. r=jaws r=bz r=joe
--HG--
rename : toolkit/themes/pinstripe/global/TopLevelImageDocument.css => toolkit/themes/pinstripe/global/media/TopLevelImageDocument.css
rename : toolkit/themes/pinstripe/global/TopLevelVideoDocument.css => toolkit/themes/pinstripe/global/media/TopLevelVideoDocument.css
rename : toolkit/themes/winstripe/global/TopLevelImageDocument.css => toolkit/themes/winstripe/global/media/TopLevelImageDocument.css
rename : toolkit/themes/winstripe/global/TopLevelVideoDocument.css => toolkit/themes/winstripe/global/media/TopLevelVideoDocument.css
2012-10-05 14:44:11 -07:00
Josh Matthews
d47a117592 Bug 792821 - Add more informative documentation to getImgCacheForDocument. r=joe 2012-09-25 23:51:18 -04:00
Ryan VanderMeulen
d3510ae5f7 Backout e5269e137290, (bug 794606), e94dbc151121 (bug 794602), and 0ea0cfc7f6f4 (bug 792821) due to mochitest-other orange on a CLOSED TREE. 2012-10-04 19:34:19 -04:00
Josh Matthews
f58ee6c31e Bug 792821 - Add more informative documentation to getImgCacheForDocument. r=joe 2012-09-25 23:51:18 -04:00
Jeff Muizelaar
fb8c5e5cd5 Bug 792199. Only decode visible images immediately when switching to a tab. r=joe
This creates a new method 'StartDecoding' which does a RequestDecode and some
decoding of the image.

--HG--
extra : rebase_source : 89c1c2f40e3dfb6e025939ebf3257f81ce429501
2012-10-04 16:02:15 -04:00
Jeff Muizelaar
1e809c4aa8 Bug 792954. Add a WeakPtr implementation to use instead of nsISupportsWeakReference. r=joe,ehsan,Waldo
This patch also replaces the usage of nsISupportsWeakReference in RasterImage as an example.

--HG--
extra : rebase_source : ac6a039dcc3227a04ac4c2221f38856bb308c695
2012-10-04 15:45:07 -04:00
Jeff Muizelaar
4f4ff0b9d8 Bug 566370. Remove return type from AddProxy(). r=joe
No one checks it, and it's not really useful.

--HG--
extra : rebase_source : 164f3c18f707551670af1acd4d735f705903a9d2
2012-10-03 18:39:59 -04:00
Josh Matthews
8d8ddeb184 Bug 795571 - Update nsIChannel consumers concerned with privacy status to use the correct NS_UsePrivateBrowsing. r=ehsan 2012-10-03 18:43:26 -04:00
Jeff Muizelaar
b76cc85271 Bug 662246. Log image url in profiles during decoding. r=joe
--HG--
extra : rebase_source : 83a418d1b8b5dbb2d5f6bfe2725f04d0459cfca0
2012-10-03 16:36:26 -04:00
Joe Drew
9f93c7b069 Back out 134f5b1d6d50 for leaking at shutdown on a CLOSED TREE 2012-10-03 16:31:28 -04:00
Jeff Muizelaar
c3e16c5475 Bug 794884. Make SupportImageWithMimeType return a bool. r=joe.
This cleans things up quite a bit.
---
 content/base/src/nsObjectLoadingContent.cpp |    4 +---
 docshell/base/nsWebNavigationInfo.cpp       |    4 +---
 image/src/imgLoader.cpp                     |    7 ++-----
 image/src/imgLoader.h                       |    2 +-
 layout/build/nsContentDLF.cpp               |    4 +---
 5 files changed, 6 insertions(+), 15 deletions(-)
2012-10-03 16:17:47 -04:00
Jeff Muizelaar
af82a52095 Bug 794884. Make SupportImageWithMimeType a static function. r=joe,r=bz
This removes supportImageWithMimeType from the idl and makes all the
callers call the static function directly. To do this we also need
to export imgLoader.h and imgRequest.h.
---
 content/base/src/nsObjectLoadingContent.cpp |    9 ++-------
 docshell/base/nsWebNavigationInfo.cpp       |    8 +++++---
 docshell/base/nsWebNavigationInfo.h         |    5 -----
 image/public/imgILoader.idl                 |    9 +--------
 image/src/Makefile.in                       |    4 ++++
 image/src/imgLoader.cpp                     |    2 +-
 image/src/imgLoader.h                       |    1 +
 layout/build/nsContentDLF.cpp               |    5 ++---
 8 files changed, 16 insertions(+), 27 deletions(-)
2012-10-03 16:17:40 -04:00
Jeff Muizelaar
89405c97c8 Bug 794884. Avoid unecessarily including imgStatusTracker.h. r=joe
---
 image/src/imgRequest.h |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
2012-10-03 16:17:26 -04:00
Joe Drew
3b864e9976 Bug 796174 - Only use the high-quality downscaler on downscales, not 1.0 scales. r=jlebar 2012-10-03 14:29:42 -04:00
Joe Drew
a8b5aa2d73 Bug 795940 - Always hold on to a reference to our image from a ScaleRequest so it can't delete itself while we're outstanding. r=jlebar 2012-10-03 14:29:33 -04:00
Joe Drew
481b289d62 Backout 629c399c6f8c for breaking the tree 2012-10-03 13:37:45 -04:00
Joe Drew
bb1da39f0e Backout d13f5f15df86 for busting the tree 2012-10-03 13:37:14 -04:00
Joe Drew
af79edbfe3 Bug 796174 - Only use the high-quality downscaler on downscales, not 1.0 scales. r=jlebar 2012-10-03 13:27:46 -04:00
Joe Drew
4e1c276d8c Bug 795940 - Always hold on to a reference to our image from a ScaleRequest so it can't delete itself while we're outstanding. r=jlebar 2012-10-03 13:26:02 -04:00
Ed Morley
c20a54b23f Bug 761049 - Disable browser_bug666317.js on Linux for frequent failures that cause havoc in subsequent tests 2012-10-03 11:49:40 +01:00
Tom Schuster
e1dc8d9cc1 Bug 795442 - Allow nsIChannel.contentDispositionFilename to be writable. r=jduell, sr=biesi 2012-10-02 09:43:00 -07:00
Matt Woodrow
10c1bdf174 Bug 795722 - Clear our cached ImageContainer when the imgFrame changes. r=joe 2012-10-02 17:01:25 +13:00
Joe Drew
86b407c027 Bug 486918 - Create and obey a high-quality downscaling pref, and turn it off on OS X and mobile. r=jlebar 2012-09-27 14:12:56 -04:00
Tatiana Meshkova
1905ac69f9 Bug 486918. Part 2: Add the ability to pre-downscale using a high-quality scaler on a separate thread. r=joe,jlebar 2012-04-03 14:57:22 -07:00
Ms2ger
372fb21863 Bug 795862 - Back out bug 486918 for build bustage in unreviewed code. 2012-10-01 14:16:03 +02:00
Joe Drew
3a386776a9 Bug 486918 - Create and obey a high-quality downscaling pref, and turn it off on OS X and mobile. r=jlebar 2012-09-27 14:12:56 -04:00
Tatiana Meshkova
6c58b4dd5f Bug 486918. Part 2: Add the ability to pre-downscale using a high-quality scaler on a separate thread. r=joe,jlebar 2012-04-03 14:57:22 -07:00
Joe Drew
02a5ccdfef Bug 786444 - Part 2 - Make imgFrame's lock status a count, not a boolean, so you can lock it multiple times and still be correct. r=jlebar
Also add a bunch of NS_ABORT_IF_FALSEs that verify that the lock status is
always valid.
2012-09-26 11:33:06 -04:00
Joe Drew
80ebba0430 Bug 786444 - Part 1 - Add an RAII class to make locking imgFrame more foolproof, use it, and be sure to lock imgFrame more correctly. r=jlebar 2012-09-26 11:33:06 -04:00
Joe Drew
14a4dd6a5f Bug 786449 - Add a place to explicitly initialize RasterImage on the main thread at startup. r=jlebar 2012-09-26 11:33:06 -04:00
Ed Morley
6250aafdf0 Backout 92530b29ac24, 42e659956dc6 & 38277196c099 (bug 486918), d3b69fe15ac3 & b2dba2108722 (bug 786444), e7381ac9e30a (bug 786449) on suspicion of causing Android reftest failures on a CLOSED TREE 2012-09-28 23:00:28 +01:00
Joe Drew
7c3734fece Bug 486918 - Create and obey a high-quality downscaling pref, and turn it off on OS X and mobile. r=jlebar
--HG--
extra : rebase_source : 386bee19522e3cb8bca16ec40e3c092a52ae18d2
2012-09-27 14:12:56 -04:00
Tatiana Meshkova
730949b4e1 Bug 486918. Part 2: Add the ability to pre-downscale using a high-quality scaler on a separate thread. r=joe,jlebar
--HG--
extra : rebase_source : fa22f42c832ae57b8e221ee03c68a45d33a929d3
2012-04-03 14:57:22 -07:00
Joe Drew
f5cad470c9 Bug 786444 - Part 2 - Make imgFrame's lock status a count, not a boolean, so you can lock it multiple times and still be correct. r=jlebar
Also add a bunch of NS_ABORT_IF_FALSEs that verify that the lock status is
always valid.

--HG--
extra : rebase_source : 70cce5ed671586529e7d698e4bcdc86f5c748aa5
2012-09-26 11:33:06 -04:00
Joe Drew
cbe2dc4d36 Bug 786444 - Part 1 - Add an RAII class to make locking imgFrame more foolproof, use it, and be sure to lock imgFrame more correctly. r=jlebar
--HG--
extra : rebase_source : 255805f32aa06b3719a1442388b7b759c1c16c89
2012-09-26 11:33:06 -04:00
Joe Drew
1140caf167 Bug 786449 - Add a place to explicitly initialize RasterImage on the main thread at startup. r=jlebar
--HG--
extra : rebase_source : d23afb459bca8c36467ad2af042580a96666b886
2012-09-26 11:33:06 -04:00