[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.15.1-1414-gc69ee75
simon.fraser at apple.com
simon.fraser at apple.com
Thu Oct 29 20:34:29 UTC 2009
The following commit has been merged in the webkit-1.1 branch:
commit ef548e339b49c79a23e193aec0da9688db7a7013
Author: simon.fraser at apple.com <simon.fraser at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri Sep 25 22:01:43 2009 +0000
2009-09-25 Simon Fraser <simon.fraser at apple.com>
Reviewed by Darin Adler.
Crash with hardware accelerated rotation of a PDF image in a data URL
<rdar://problem/7250378>
PDF images don't return a color space from CGImageGetColorSpace(),
so we need to null-check the return value before use.
Test: compositing/color-matching/pdf-image-match.html
* platform/graphics/mac/GraphicsLayerCA.mm:
(WebCore::GraphicsLayerCA::setContentsToImage):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@48771 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index d259e9a..25495f8 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,17 @@
+2009-09-25 Simon Fraser <simon.fraser at apple.com>
+
+ Reviewed by Darin Adler.
+
+ Crash with hardware accelerated rotation of a PDF image in a data URL
+ <rdar://problem/7250378>
+
+ Testcase for a PDF image as the contents of a compositing layer.
+
+ * compositing/color-matching/pdf-image-match.html: Added.
+ * platform/mac/compositing/color-matching/pdf-image-match-expected.checksum: Added.
+ * platform/mac/compositing/color-matching/pdf-image-match-expected.png: Added.
+ * platform/mac/compositing/color-matching/pdf-image-match-expected.txt: Added.
+
2009-09-25 Darin Adler <darin at apple.com>
Reviewed by Geoffrey Garen.
diff --git a/LayoutTests/compositing/color-matching/pdf-image-match.html b/LayoutTests/compositing/color-matching/pdf-image-match.html
new file mode 100644
index 0000000..75adb7f
--- /dev/null
+++ b/LayoutTests/compositing/color-matching/pdf-image-match.html
@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
+ "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en">
+<head>
+ <meta http-equiv="content-type" content="text/html; charset=utf-8" />
+ <title>Rotation Test</title>
+ <style type="text/css">
+
+ img.test {
+ width: 20px;
+ height: 20px;
+ -webkit-transform: translateZ(0);
+ }
+ </style>
+</head>
+<body>
+
+<p>This test should not crash</p>
+<img class="test" src="data:application/pdf;base64,JVBERi0xLjMKJcTl8uXrp/Og0MTGCjQgMCBvYmoKPDwgL0xlbmd0aCA1IDAgUiAvRmlsdGVyIC9GbGF0ZURlY29kZSA+PgpzdHJlYW0KeAF9U7uO3DAM7PUV/IHwREqi5HqLA9Id9hMMJEiQKxZb5PczpGx5mxy8D3usIYej0YM+6EFC3/H9TZmNftHb7Sl0u1OO636jt3cAP5+x0MFvgo9u3Ei64nf/JGmZex+apAnXLoMca5x7jf9u3S7AlHtRfzNZJNY561DaSapxHVqTdOFYVNBJnL6Axqpo8coqmTfFmlX6BCDCmycpJ+tUiF6rdO0cLY7mUHEAS+FipTs8C4/25+HRc3eP1D3SjJklm98M9ZtP0sJbt6JQx5s0jHkiSTK3UTHL4hlbH1t1J4zFYjWsLQbaxqNgcSH4N5GGqX31wUo7XmWNxVfphZztT9bSuL+UrtwVYtPV/kAujS+8HwiOXzMm5YuYpCMmsAeEpjB8bBGTaQ8w4zwqbGongt2sMCyCMnlmCIUIBvWguD/IRUYerMR2wiAUuJA5KrycPLe1hEU9suLVEY0TWQoQFjf20jnDMqvXwdoLCiwFB4L0x/a/zLeTx0Xp75enp0I8HOnN/fAHQ2pb+hPkB+z937GrcQrbxhrUYn4o5+Mkf/wD8W/FKAplbmRzdHJlYW0KZW5kb2JqCjUgMCBvYmoKNDIxCmVuZG9iagoyIDAgb2JqCjw8IC9UeXBlIC9QYWdlIC9QYXJlbnQgMyAwIFIgL1Jlc291cmNlcyA2IDAgUiAvQ29udGVudHMgNCAwIFIgL01lZGlhQm94IFswIDAgMjYgMjddCj4+CmVuZG9iago2IDAgb2JqCjw8IC9Qcm9jU2V0IFsgL1BERiBdIC9Db2xvclNwYWNlIDw8IC9DczEgNyAwIFIgPj4gL0V4dEdTdGF0ZSA8PCAvR3MzIDggMCBSCi9HczEgOSAwIFIgL0dzMiAxMCAwIFIgPj4gPj4KZW5kb2JqCjggMCBvYmoKPDwgL1R5cGUgL0V4dEdTdGF0ZSAvQ0EgMSA+PgplbmRvYmoKOSAwIG9iago8PCAvVHlwZSAvRXh0R1N0YXRlIC9DQSAwLjUgPj4KZW5kb2JqCjEwIDAgb2JqCjw8IC9UeXBlIC9FeHRHU3RhdGUgL2NhIDAuMyA+PgplbmRvYmoKMTEgMCBvYmoKPDwgL0xlbmd0aCAxMiAwIFIgL04gMyAvQWx0ZXJuYXRlIC9EZXZpY2VSR0IgL0ZpbHRlciAvRmxhdGVEZWNvZGUgPj4Kc3RyZWFtCngBhVTPaxNBFP42bqnQIghaaw6yeJAiSVmraEXUNv0RYmsM2x+2RZBkM0nWbjbr7ia1pYjk4tEq3kXtoQf/gB568GQvSoVaRSjeqyhioRct8c1uTLal6sDOfvPeN+99b3bfAA1y0jT1gATkDcdSohFpbHxCavyIAI6iCUE0JVXb7E4kBkGDc/l759h6D4FbVsN7+3eyd62a0raaB4T9QOBHmtkqsO8XcQpZEgKIPN+hKcd0CN/j2PLsjzlOeXjBtQ8rPcRZInxANS3Of024U80l00CDSDiU9XFSPpzXi5TXHQdpbmbGyBC9T5Cmu8zuq2KhnE72DpC9nfR+TrPePsIhwgsZrT9GuI2e9YzVP+Jh4aTmxIY9HBg19PhgFbcaqfg1whRfEE0nolRx2S4N8Ziu/VbySoJwkDjKZGGAc1pIT9dMbvi6hwV9JtcTr+J3VlHheY8TZ97U3e9F2gKvMA4dDBoMmg1IUBBFBGGYsFBAhjwaMTSycj8jqwYbk3sydSRqu3RiRLFBezbcPbdRpN08/igicZRDtQiS/EH+Kq/JT+V5+ctcsNhW95Stm5q68uA7xeWZuRoe19PI43NNXnyV1HaTV0eWrHl6vJrsGj/sV5cx5oI1j8RzsPvxLV+VzJcpjBTF41Xz6kuEdVoxN9+fbH87PeIuzy611nOtiYs3VpuXZ/1qSPvuqryT5lX5T1718fxnzcRj4ikxJnaK5yGJl8Uu8ZLYS6sL4mBtxwidlYYp0m2R+iTVYGCavPUvXT9beL1Gfwz1UZQZzNJUifd/wipkNJ25Dm/6j9vH/Bfk94rnnygCL2zgyJm6bVNx7xChZaVuc64CF7/RffC2bmujfjj8BFg8qxatUjWfILwBHHaHeh7oKZjTlpbNOVKHLJ+TuunKYlLMUNtDUlLXJddlSxazmVVi6XbYmdMdbhyhOUL3xKdKZZP6r/ERsP2wUvn5rFLZfk4a1oGX+m/AvP1FCmVuZHN0cmVhbQplbmRvYmoKMTIgMCBvYmoKNzM3CmVuZG9iago3IDAgb2JqClsgL0lDQ0Jhc2VkIDExIDAgUiBdCmVuZG9iagozIDAgb2JqCjw8IC9UeXBlIC9QYWdlcyAvTWVkaWFCb3ggWzAgMCAyNiAyN10gL0NvdW50IDEgL0tpZHMgWyAyIDAgUiBdID4+CmVuZG9iagoxMyAwIG9iago8PCAvVHlwZSAvQ2F0YWxvZyAvUGFnZXMgMyAwIFIgL1ZlcnNpb24gLzEuNCA+PgplbmRvYmoKMTQgMCBvYmoKKE1hYyBPUyBYIDEwLjYuMSBRdWFydHogUERGQ29udGV4dCkKZW5kb2JqCjE1IDAgb2JqCihEOjIwMDkwOTI0MDM1OTM1WjAwJzAwJykKZW5kb2JqCjEgMCBvYmoKPDwgL1Byb2R1Y2VyIDE0IDAgUiAvQ3JlYXRpb25EYXRlIDE1IDAgUiAvTW9kRGF0ZSAxNSAwIFIgPj4KZW5kb2JqCnhyZWYKMCAxNgowMDAwMDAwMDAwIDY1NTM1IGYgCjAwMDAwMDIwMjkgMDAwMDAgbiAKMDAwMDAwMDUzNiAwMDAwMCBuIAowMDAwMDAxNzkwIDAwMDAwIG4gCjAwMDAwMDAwMjIgMDAwMDAgbiAKMDAwMDAwMDUxNyAwMDAwMCBuIAowMDAwMDAwNjM4IDAwMDAwIG4gCjAwMDAwMDE3NTQgMDAwMDAgbiAKMDAwMDAwMDc1NyAwMDAwMCBuIAowMDAwMDAwODAxIDAwMDAwIG4gCjAwMDAwMDA4NDcgMDAwMDAgbiAKMDAwMDAwMDg5NCAwMDAwMCBuIAowMDAwMDAxNzM0IDAwMDAwIG4gCjAwMDAwMDE4NzEgMDAwMDAgbiAKMDAwMDAwMTkzNSAwMDAwMCBuIAowMDAwMDAxOTg3IDAwMDAwIG4gCnRyYWlsZXIKPDwgL1NpemUgMTYgL1Jvb3QgMTMgMCBSIC9JbmZvIDEgMCBSIC9JRCBbIDxmN2FjMTg4NTA2ZTVhNzEwMGY5M2RiZGExYWJiN2IwNT4KPGY3YWMxODg1MDZlNWE3MTAwZjkzZGJkYTFhYmI3YjA1PiBdID4+CnN0YXJ0eHJlZgoyMTA0CiUlRU9GCjEgMCBvYmoKPDwvQXV0aG9yIChDb2x0ZXIgUmVlZCkvQ3JlYXRpb25EYXRlIChEOjIwMDkwOTI0MDMzNzAwWikvQ3JlYXRvciAoT21uaUdyYWZmbGUgUHJvZmVzc2lvbmFsIDUuMS4xKS9Nb2REYXRlIChEOjIwMDkwOTI0MDM1OTAwWikvUHJvZHVjZXIgMTQgMCBSID4+CmVuZG9iagp4cmVmCjEgMQowMDAwMDAyNTgxIDAwMDAwIG4gCnRyYWlsZXIKPDwvSUQgWzxmN2FjMTg4NTA2ZTVhNzEwMGY5M2RiZGExYWJiN2IwNT4gPGY3YWMxODg1MDZlNWE3MTAwZjkzZGJkYTFhYmI3YjA1Pl0gL0luZm8gMSAwIFIgL1ByZXYgMjEwNCAvUm9vdCAxMyAwIFIgL1NpemUgMTY+PgpzdGFydHhyZWYKMjc0MQolJUVPRgo="/>
+
+</body>
+</html>
diff --git a/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.checksum b/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.checksum
new file mode 100644
index 0000000..e6cd903
--- /dev/null
+++ b/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.checksum
@@ -0,0 +1 @@
+aab69a7ec65adfef337cdc8c5fa5a66b
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.png b/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.png
new file mode 100644
index 0000000..a91a8aa
Binary files /dev/null and b/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.png differ
diff --git a/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.txt b/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.txt
new file mode 100644
index 0000000..275b507
--- /dev/null
+++ b/LayoutTests/platform/mac/compositing/color-matching/pdf-image-match-expected.txt
@@ -0,0 +1,14 @@
+layer at (0,0) size 800x600
+ RenderView at (0,0) size 800x600
+layer at (0,0) size 800x82
+ RenderBlock {HTML} at (0,0) size 800x82
+ RenderBody {BODY} at (8,16) size 784x58
+ RenderBlock {P} at (0,0) size 784x18
+ RenderText {#text} at (0,0) size 160x18
+ text run at (0,0) width 160: "This test should not crash"
+ RenderBlock (anonymous) at (0,34) size 784x24
+ RenderText {#text} at (0,0) size 0x0
+ RenderText {#text} at (0,0) size 0x0
+ RenderText {#text} at (0,0) size 0x0
+layer at (8,50) size 20x20
+ RenderImage {IMG} at (0,0) size 20x20
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 7aff711..407cc6b 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2009-09-25 Simon Fraser <simon.fraser at apple.com>
+
+ Reviewed by Darin Adler.
+
+ Crash with hardware accelerated rotation of a PDF image in a data URL
+ <rdar://problem/7250378>
+
+ PDF images don't return a color space from CGImageGetColorSpace(),
+ so we need to null-check the return value before use.
+
+ Test: compositing/color-matching/pdf-image-match.html
+
+ * platform/graphics/mac/GraphicsLayerCA.mm:
+ (WebCore::GraphicsLayerCA::setContentsToImage):
+
2009-09-25 Darin Adler <darin at apple.com>
Reviewed by Geoffrey Garen.
diff --git a/WebCore/platform/graphics/mac/GraphicsLayerCA.mm b/WebCore/platform/graphics/mac/GraphicsLayerCA.mm
index 2afcea4..e9960f1 100644
--- a/WebCore/platform/graphics/mac/GraphicsLayerCA.mm
+++ b/WebCore/platform/graphics/mac/GraphicsLayerCA.mm
@@ -704,7 +704,7 @@ void GraphicsLayerCA::setContentsToImage(Image* image)
CGColorSpaceRef colorSpace = CGImageGetColorSpace(m_pendingContentsImage.get());
static CGColorSpaceRef deviceRGB = CGColorSpaceCreateDeviceRGB();
- if (CFEqual(colorSpace, deviceRGB)) {
+ if (colorSpace && CFEqual(colorSpace, deviceRGB)) {
// CoreGraphics renders images tagged with DeviceRGB using the color space of the main display. When we hand such
// images to CA we need to tag them similarly so CA rendering matches CG rendering.
static CGColorSpaceRef genericRGB = CGDisplayCopyColorSpace(kCGDirectMainDisplay);
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list