Commit Graph

2095 Commits

Author SHA1 Message Date
Christoph Kerschbaumer
563d5a0864 Bug 1256999 - Use nsIDocument for ImageCacheKey. r=bz r=seth 2016-04-11 11:00:03 +02:00
Christoph Kerschbaumer
52a7f283b3 Bug 1256999 - Pass the right context to new channels for image loads. r=bz r=seth 2016-04-11 10:58:03 +02:00
Timothy Nikkel
28a90df677 Bug 1262269. Re-enable some imagelib tests that are passing. 2016-04-10 19:21:48 -05:00
Ryan VanderMeulen
f2ae6bedc6 Backed out changeset 98aa2df1a69f (bug 1262269) for frequent Linux failures in the re-enabled tests. 2016-04-10 21:52:07 -04:00
Timothy Nikkel
7bd583a54c Bug 1262269. Re-enable some imagelib tests that are passing. 2016-04-10 19:21:48 -05:00
Timothy Nikkel
909b97b554 Bug 1263058. Fix image/test/mochitest/test_animation2.html to test what it is intending to test. r=mstange
opacity >= 0.999 gets interpreted as 1 for painting purposes, so does not serve the purpose of creating a container layer. Use will-change instead.
2016-04-08 01:13:53 -05:00
Kartikaya Gupta
76e300e3c3 Bug 1261158 - Convert test_bug1132427.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 4NXUjL4uGEP
2016-04-04 16:28:13 -04:00
Kartikaya Gupta
146d7600ec Bug 1261158 - Convert test_xultree_animation.xhtml to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: BDanrpo7ES
2016-04-04 16:28:13 -04:00
Kartikaya Gupta
372512629b Bug 1261158 - Convert test_synchronized_animation.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: LrftftKDGn6
2016-04-04 16:28:12 -04:00
Kartikaya Gupta
e19f2b345c Bug 1261158 - Convert test_svg_filter_animation.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: K5ZxnkhR6nE
2016-04-04 16:28:12 -04:00
Kartikaya Gupta
e13c70fd27 Bug 1261158 - Convert test_svg_animatedGIF.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: IIJT4178POO
2016-04-04 16:28:12 -04:00
Kartikaya Gupta
e2bbe3839c Bug 1261158 - Convert test_staticClone.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 8CUzA0EzM2G
2016-04-04 16:28:12 -04:00
Kartikaya Gupta
a11b57dc20 Bug 1261158 - Convert test_removal_onload.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: DL4IQaUl7t5
2016-04-04 16:28:12 -04:00
Kartikaya Gupta
f3c53ca5e1 Bug 1261158 - Convert test_removal_ondecode.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 6YOHnVhRqkY
2016-04-04 16:27:59 -04:00
Kartikaya Gupta
9a87fbceec Bug 1261158 - Convert test_net_failedtoprocess.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: HO1BMl760MQ
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
d278cd5ea7 Bug 1261158 - Convert test_has_transparency.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 9tR3LH4wg4U
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
4f6713a578 Bug 1261158 - Convert test_changeOfSource2.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 9mdR16WzTkU
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
c11fae07bc Bug 1261158 - Convert test_changeOfSource.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: BqYNvBW06ko
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
c79e0934bb Bug 1261158 - Convert test_bullet_animation.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 8kgi3p2DMkw
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
91f2e806c0 Bug 1261158 - Convert test_background_image_anim.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: 5RJwrm5RKiU
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
32f3c750d7 Bug 1261158 - Convert test_animSVGImage2.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: JT02zkq1JJm
2016-04-04 16:27:58 -04:00
Kartikaya Gupta
637d1ef3d8 Bug 1261158 - Convert test_animSVGImage.html to a mochitest-plain. r=tnikkel
MozReview-Commit-ID: E2n2JM0PjzR
2016-04-04 16:27:57 -04:00
Timothy Nikkel
6aeb6d5d05 Bug 1260247. In chaos mode randomly decide to start a new image load even if the image is in the image cache when allowed by spec. r=seth
If the image load is from the same document that cached the image we are required to use the cached version. Otherwise we should be free to ignore the cached version.
2016-04-02 13:53:12 -05:00
Timothy Nikkel
b0b9bfd047 Bug 1222596. If RasterImage::LookupFrame does (some) sync decoding and encouters an error we don't want to return the surface with an error. r=seth
If the sync decoding the LookupFrame does encounters an error it will set mError on the RasterImage, which LookupFrame callers check before calling LookupFrame. But they've called LookupFrame before the error was encountered, so we check if the frame has had Abort called on it to determine if we should return it at all.

We only does this if one of the sync decode flags was passed in because IsAborted needs to get the imgFrame's monitor, so we don't want to block consumers that haven't asked for decoding.
2016-04-01 12:44:17 -05:00
Seth Fowler
603db7a389 Bug 1209780 (Part 1) - Mark DrawResult MOZ_MUST_USE. r=tn 2015-10-25 13:20:13 -07:00
Timothy Nikkel
5afe8e5453 Bug 1257101. imgFrame::IsImageComplete says whether we've had pixels decoded to the whole image rect, but it's used to check if the frame is finished decoding. These are different things when the image has more than one progress pass. r=seth
This means that in RasterImage::LookupFrame when we are asked to do a sync decode (if needed) we use WaitUntilComplete to wait until the frame is finished decoding.  But we would actually return after the next progressive pass notified the monitor to wake up. Thus, we would draw a not-fully-decoded image even though the sync decode flag was passed.

The change in FrameAnimator means that we won't draw the next frame in an animated image until all progressive passes of that image are complete. This seems like what we want anyways.

There is one real use of IsImageComplete left, in imgFrame::Draw, where we need to know if the decoded image data covers the whole image frame. (There are a couple of uses of IsImageComplete in asserts.)
2016-03-23 19:31:42 -05:00
Timothy Nikkel
28d73a34c5 Backed out changeset bc854d1c6ab7 (bug 1209780) for statis analysis failures on OS X. 2016-03-21 16:29:15 -05:00
Timothy Nikkel
417ecff3c2 Bug 1210745. Change image progress asserts to allow transparency to be posted after the size is posted. r=seth
The PNG decoder posts the size almost immediately, and later posts transparency (even for non-animated images).

It would be nice to still assert what this assert is intending (that transparency of non-animated images is posted during the metadata decode) but we don't have any easy way of telling when a metadata finishes here.
2016-03-21 16:15:20 -05:00
Seth Fowler
3822e4c6c4 Bug 1210745 - Update CheckProgressConsistency() to match current ImageLib behavior. r=tn 2016-02-25 13:12:58 -06:00
Timothy Nikkel
393d0a8b07 Bug 1225934. Better describe the effect of the flag FLAG_HIGH_QUALITY_SCALING. Comment only change. 2016-03-21 16:15:20 -05:00
Seth Fowler
8b46f4d5f2 Bug 1209780 (Part 1) - Mark DrawResult MOZ_MUST_USE. r=tn 2015-10-25 13:20:13 -07:00
Geoff Brown
f8b671c940 Bug 1251013 - Update AndroidVersion annotations in test manifests - xpcshell; r=jmaher 2016-03-17 10:00:50 -06:00
Geoff Brown
29977660c0 Bug 1251013 - Update android_version annotations in test manifests - mochitest; r=jmaher 2016-03-17 10:00:48 -06:00
Mike Hommey
0a771514a8 Bug 1256988 - Replace tests on MOZ_WIDGET_GTK with tests on MOZ_WIDGET_TOOLKIT containing gtk. r=chmanchester
But only do that for the AC_SUBST. The AC_DEFINE is still useful.
2016-03-17 06:49:44 +09:00
Timothy Nikkel
cc2e20ef2c Bug 1255675. Add very basic reftests that test our downscale during decode code for every image format we support.
Some fuzz is necessary of course, but these tests catch this bug, so they should be successful in ensuring basically functionality isn't regressed.
2016-03-16 14:10:31 -05:00
Edwin Flores
0d5d29eb98 Bug 1234077 - Test that GIFs are still drawn correctly when their frame rects are truncated - r=seth 2016-03-15 10:31:31 +00:00
Sebastian Hengst
17ed0683ac Backed out changeset e7b960c2c8e9 (bug 1234077) for reftest failures. r=backout 2016-03-14 21:08:53 +01:00
Timothy Nikkel
ce9ddc5a29 Bug 1255675. SurfacePipe handles removing the frame rect, so pass the frame rect directly.
This is left over from the pre-SurfacePipe code that interacted directly with the Downscaler. It was calculating the size of the surface for the Downscaler to use, and then the Downscaler would handle putting transparent pixels inside that surface (and outside the framerect).

--HG--
extra : rebase_source : aad384fa8589f291254f0a18537a5d6674487182
2016-03-13 15:56:00 -07:00
Edwin Flores
3a94cf4328 Bug 1234077 - Test that GIFs are still drawn correctly when their frame rects are truncated - r=seth 2016-03-14 18:26:38 +00:00
Bogdan Postelnicu
c9c042d934 Bug 1230907 - mPixelRowSize added to constructor with 0 value. r=seth 2016-03-14 10:36:36 +02:00
Seth Fowler
c987885db6 Bug 1255104 - Use SurfacePipe in nsIconDecoder. r=njn 2016-03-10 18:45:29 -08:00
Seth Fowler
3978c77cb6 Bug 1247152 (Part 2) - Remove even more code from the GIF decoder. r=edwin 2016-03-09 15:39:02 -08:00
Seth Fowler
875464424a Bug 1247152 (Part 1) - Use SurfacePipe in the GIF decoder. r=njn 2016-03-09 15:39:02 -08:00
CJKu
1c893a0ddf Bug 619500: Part 4. Remove one unused data member in SVGDrawingParameters; r=dholbert
MozReview-Commit-ID: EN3HVuwTm7y
2016-03-08 15:54:14 +08:00
CJKu
d57efebb5f Bug 619500: Part 2. When drawing an SVG image as a CSS border-image, use preverveAspectRatio="none"; r=dholbert
MozReview-Commit-ID: JH5E3An4opJ
2016-03-08 15:54:13 +08:00
CJKu
19414f3998 Bug 619500: Part 1. Default sizing for specified size of SVG images which have no constraints; r=dholbert r=seth
MozReview-Commit-ID: 8DI86w6Ni8T
2016-03-08 15:54:13 +08:00
Seth Fowler
b935d483a8 Bug 1225934 - Never allow surface substitution when FLAG_HIGH_QUALITY_SCALING is disabled. r=tn 2016-03-07 17:17:16 -08:00
Seth Fowler
5a675c9ccb Bug 1251806 - In RasterImage::GetFrameInternal(), check if the frame covers the actual surface size rather than the requested surface size. r=tn 2016-03-07 17:17:16 -08:00
Seth Fowler
6451b96c1d Bug 1251807 - Use the surface's size, not the intrinsic size, in CopyFrame. r=tn 2016-03-07 17:15:25 -08:00
Timothy Nikkel
c2cd155669 Bug 1253362. SVGDocumentWrapper::IsAnimated can be called after SVGDocumentWrapper::DestroyViewer so null check mViewer. r=dholbert
The testcase has an svg-as-image inside an svg-as-image. At shutdown the viewer of the inner svg-as-image is destroyed (via the shutdown observer) first. Then the outer svg-as-image destroys its viewer which tries to unregister all image requests from the refresh driver. So it unregisters the inner svg-as-image, which calls GetAnimated.
2016-03-06 14:59:19 -06:00