[webkit-changes] [139256] trunk

Wed, 09 Jan 2013 16:43:19 -0800

Title: [139256] trunk
Revision
139256
Author
ta...@google.com
Date
2013-01-09 16:44:57 -0800 (Wed, 09 Jan 2013)

Log Message

border-radius with box-shadow is not rendered correctly
https://bugs.webkit.org/show_bug.cgi?id=106404

Reviewed by Hajime Morita.

Source/WebCore:

RoundedRect::isRenderable is wrong. So RoundedRect::adjustRadii
is invoked for renderable RoundedRects.

Test: fast/borders/border-radius-with-box-shadow.html

* platform/graphics/RoundedRect.cpp:
(WebCore::RoundedRect::isRenderable):
Have to compare topLeft's height plus bottomLeft's height with
rect's height and to compare topRight's height plus bottomRight's
height with rect's height.

LayoutTests:

* fast/borders/border-radius-with-box-shadow-expected.txt: Added.
* fast/borders/border-radius-with-box-shadow.html: Added.
* platform/chromium-linux/fast/borders/border-radius-with-box-shadow-expected.png: Added.

Modified Paths

Added Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (139255 => 139256)


--- trunk/LayoutTests/ChangeLog	2013-01-10 00:39:55 UTC (rev 139255)
+++ trunk/LayoutTests/ChangeLog	2013-01-10 00:44:57 UTC (rev 139256)
@@ -1,3 +1,14 @@
+2013-01-09  Takashi Sakamoto  <ta...@google.com>
+
+        border-radius with box-shadow is not rendered correctly
+        https://bugs.webkit.org/show_bug.cgi?id=106404
+
+        Reviewed by Hajime Morita.
+
+        * fast/borders/border-radius-with-box-shadow-expected.txt: Added.
+        * fast/borders/border-radius-with-box-shadow.html: Added.
+        * platform/chromium-linux/fast/borders/border-radius-with-box-shadow-expected.png: Added.
+
 2013-01-09  Anders Carlsson  <ander...@apple.com>
 
         Update a test result.

Added: trunk/LayoutTests/fast/borders/border-radius-with-box-shadow-expected.txt (0 => 139256)


--- trunk/LayoutTests/fast/borders/border-radius-with-box-shadow-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/fast/borders/border-radius-with-box-shadow-expected.txt	2013-01-10 00:44:57 UTC (rev 139256)
@@ -0,0 +1 @@
+

Added: trunk/LayoutTests/fast/borders/border-radius-with-box-shadow.html (0 => 139256)


--- trunk/LayoutTests/fast/borders/border-radius-with-box-shadow.html	                        (rev 0)
+++ trunk/LayoutTests/fast/borders/border-radius-with-box-shadow.html	2013-01-10 00:44:57 UTC (rev 139256)
@@ -0,0 +1,26 @@
+<!doctype html>
+<head>
+<style>
+.fo_menu{
+    width: 549px;
+    height: 45px;
+    background-color: #000000;
+    border-top-left-radius: 130px 70px;
+    border-top-right-radius: 130px 70px;
+    margin:50ppx;
+    box-shadow: 0px 0px 1px 1px #000000;
+}
+</style>
+<script>
+if (window.testRunner)
+   testRunner.dumpAsText(true);
+</script>
+</head>
+<body>
+<!-- [bug 106404] https://bugs.webkit.org/show_bug.cgi?id=106404 -->
+<!-- border-radius with box-shadow is not rendered correctly. -->
+<!-- This test is for checking whether the above bug is correctly -->
+<!-- fixed or not. -->
+<div class="fo_menu">
+</div>
+</body>

Added: trunk/LayoutTests/platform/chromium-linux/fast/borders/border-radius-with-box-shadow-expected.png (0 => 139256)


--- trunk/LayoutTests/platform/chromium-linux/fast/borders/border-radius-with-box-shadow-expected.png	                        (rev 0)
+++ trunk/LayoutTests/platform/chromium-linux/fast/borders/border-radius-with-box-shadow-expected.png	2013-01-10 00:44:57 UTC (rev 139256)
@@ -0,0 +1,12 @@
+\x89PNG
+
+
+IHDR X')tEXtchecksuma22288c0f6dc62ecd5b163b292ddcfbd\xDC\xCE\xD4IDATx\x9C\xED\xDDKo\x9Cg\xFD\xC7\xE1xb{\\xDA\xE0\x98\x92\xD4\xB4$*\xE2 RE\xADXTH *\xED\x82\xBC^\xB2 \xEDP$\x8DZ\x81\x84I\x92AU\x8AR5\xD4DIH;1Nl'\x8F\xEB9=\xF3_XͿ\x97\x9E\xBE\x93I\xC6׵\xF4i~#Y\xF6g\xEE\xE7\x9E\xFB\xE9k\xB7\xDB;\xC8)u{\x80^#\xB0\xC2@\x98\xC0Xa\xFD\xDD\xA0\x83\xDA\xEDv\xADV+\x8A\xA2(\x8AZ\xAD\xD6n\xB7\x8B\xA2h4\xCDf\xF3\xCE\xD74\x9B\xCDF\xA3QE\xE7\xF8\xA5Ri``\xA0\xBF\xFF\xFF\xFFD\xF7\xF7\xF7\x94J\xA5\xBE\xBE\xBEr\xB9\*\x95J\xA5R\xB9\\xEE\xEB\xEB\xEB\xE2\x9C\xC0\xBB\xE9\x91\xC0j4\x8DF\xA3V\xAB5\x9B\xCDz\xBD\xDEh4666666\xE6\xE7\xE7k\xB5Z\xADV\xBBr\xE5J\xADVk4\x8B\x8B\x8Bkkk\x9BEU\xC5\xDA\xDA\xDA\xE2\xE2\xE2;\x93\xA0\xEB\xFA\xFB\xFB\xF7\xEC\xD93::Z*\x95v\xEC\xD8Q*\x95FGG\xF7\xEC\xD9300P.\x97\xF7\xED\xDBW.\x97\xCB\xE5\xF2\xD4\xD4\xD4\xD0\xD0\xD0\xD0\xD0\xD0\xC0\xC0\xC0\xE0\xE0`\xB9\\xE8\xF63\x80\xED\xEE\xBE\xACv\xBB]\xAF\xD77޶\xBE\xBE~\xFD\xFA\xF5\xFF\xFB\xDF+++׮][^^~\xF3\xCD7766n޼YE\xBBݮV\xAB\xEDv\xBB\xDDn\xFF\xC7z\xD5\xE6g\xBB\xF8\\xB6\xD4\xD7׷YW\x9B6״\xFA\xFA\xFA\xFA\xFA\xFAFFF6?\xBBk׮\xA1\xA1\xA1O}\xEAS\xBBw\xEF޻w\xEF\xF8\xF8\xF8\xE4\xE4\xE4'>\xF1\x89\x8F\xFC\xE3\xC3\xC3\xC3Co\xB4\xD0wY\xDF\xFD\x92\xADVk}}}}}}cc\xE3\xF6\xED\xDB\xF3\xF3\xF3\x97\xDEv\xE5ʕ\xA5\xA5\xA5j\xB5\xBA\xBE\xBE\xDEj\xB5\xEA\xF5z\xAB\xD5*\x95J֥\x80\x9E\xD7\xDF\xDF_\xC5Ν;w\xEE\xDC9<<<22\xF2\xF0\xC3\xEF۷\xEFѷMMM=\xF8\xE0\x83CCC\xC3\xC3\xC3\xC3\xC3\xC3;w\xEE\xEC\xF6\xD4\xD0\xFB\xEE\xE9\xC0j\xB7\xDBo\xBD\xF5V\xB5Z\xADT*KKK\xE7Ν{\xE3\x8D7Ο?\xE1…7n\xAC\xAD\xADm^\xFE\xEB\xEB\xEB\xD3Rw\xF4\xF7\xF7\xB7\xDB\xED\xCDˈ\xA3\xA3\xA3\xFB\xF7\xEF?p\xE0\xC0g?\xFB\xD9\xC7\xFC\xE1\x87y\xE0\x81\xAClA\x87܋\x81\xD5l67\xA3jee\xE5ܹs\xAF\xBF\xFE\xFA\xEC\xEC\xEC\x85\xEEl\x9F\xAA\xD7\xEBݞ\xE0~288xg#\xD7\xFE\xFD\xFB\x9Fx\xE2\x89/|\xE1\x8F?\xFE\xF8\xF8\xF8\xF8fl\xBDsC=\xF0\xD1\xDDC\x81\xB5\xD9U\xB7n\xDDZXX\x98\x9D\x9D=u\xEA\xD4\xDF\xFF\xFE\x{1C5145}J\xA5R*\x95*\x95J\xB7\xE8cccEQ\x8C\x8D\x8DMNN~\xF1\x8B_|ꩧ\x9Ex\xE2\x89\xC9\xC9ɇzHiAJ\xF7뿻\xEA\xB5\xD7^\x9B\x9F\x9Fo6\x9B\xA2
+\xA0\xA3\xC6\xC6\xC6\xFA\xFB\x{39E699E}|\xF2I\xA5A\xDD\xACZ\xAD\xB6\xBC\xBC|\xE5ʕӧO\xDF骾\xBE\xBE\x95\x95\x95n\x8D\xB0=\x8D\x8F\x8F\xB7\xDB\xED;\xA5u\xE8С}\xFB\xF6\xED޽\xBB\.w{4\xB8/u!\xB0\x8A\xA2\xB8\xB3\xBFjzz\xFA\xAF\xFD\xEB?\xFF\xF9\xCF;v\xE8*\x80\xAE߱c\xC7g>\xF3\x99/}\xE9K\xCF<\xF3̝}Z\xEF<3xOw5\xB0\x8D\xC6͛7\xAF]\xBB\xF6\xCA+\xAF\xFC\xE5/9u\xEAԭ[\xB7*\x95\xCA\xC6\xC6\xC6]\x9B\x80\xF7444\xB4k׮\xD1\xD1ѧ\x9Ez\xEA\xCB_\xFE\xF2\xD3O?\xBDw\xEF\xDE]\xBBv9\xC2ާ\xBBX\x8DFcuu\xF5…'N\x9C\xF8\xE3\xFF\xF8\x8F\xFC\xA3\./,,܅\x87\xE0C\x9B\x9C\x9C\xAC\xD5j\x9F\xFB\xDC\xE7\xBE\xF6\xB5\xAF}\xE3\xDFؿ\xFF\xC7>\xF61\x99\xEF\xA9\xE3\x81\xF5δ:~\xFC\xF8\xF9\xF3\xE7k\xB5\xDA͛7;\xFA\xA0\xEDڵ\xAB\.8p\xE0\x9B\xDF\xFC\xA6̂\x{1E30C1}\xD5j\xB5VVVΟ?\xBF\x99Vkkk7n\xDC\xE8\xD0\xC3\xD0Q\xA3\xA3\xA3\x93\x93\x93\x9B\x99u\xE0\xC0\x81\xF1\xF1q\xE7\xC2Ö:X\xEDv{mm\xED\xE2ŋ\xBF\xFD\xEDo\xA7\xA7\xA7\xA5@ϸ\x93Y\xCF<\xF3\xCCw\xBE\xF3\x9DO\xFAӣ\xA3\xA3N\x84\x87\xFF\x90\xAC\x8D\x8D\x8D\xCDm\xEC\xBF\xF9\xCDoN\x9F>]\xAFץ@\x8F\x99\x98\x98<t\xE8\xD0\xF8\xC3\xCD-\xF0CCC\xDD
+\xEE!\xC9\xC0j\xB5Z7n\xDC8s\xE6̋/\xBE\xF8\x87?\xFC\xA1(\x8A+W\xAE\xA4~8\xF7\x9A}\xFB\xF6
+|\xFD\xEB_\xFF\xC1~p\xF0\xE0\xC1\x89\x89	WaS&\xB06\xEF\xCA|\xE1…\x97_~\xF9\xE5\x97_^^^\xFE׿\xFE\xF5\xD1,\xF7\xBE\xC7{l\xF7\xEE\xDD\xDF\xFD\xEEw\xBF\xF7\xBD\xEF\xED߿\xFF\x81\xE8\xF6D\xD0}\x81\xC0*\x8Abyy\xF9\x95W^9z\xF4\xE8ɓ'WVV\xD6\xD7\xD7#\xC3p_\xFA\xE9\xA7\xF4\xA3=\xF9\xE4\x93\xE3\xE3\xE3&e\x9B\xFB\xA8\x81U\xAF\xD7/_\xBE\xFC\xFB\xDF\xFF\xFEW\xBF\xFAյk\xD7\xE6\xE7\xE7S\x93p\x99\x9A\x9A\x9A\x9C\x9C|\xF6\xD9g\xBF\xF5\xADo}\xF2\x93\x9F\xEC\xF6D\xD05>\xB0\xDA\xEDv\xB5Z=s\xE6̯\xFD\xEB\xDF\xFD\xEEwkkkN\xB7\xD8\xE66\xCF\xFF\xF6\xB7\xBF\xFD\xDCs\xCF<xpdd\xC4ٞ>d`\xB5\xDB\xED\xD5\xD5ՙ\x99\x99#G\x8E\x9C>}zqq1>\xF7\xA9={\xF6:t\xE8\xC7?\xFE\xF1W\xBF\xFAՇzHc\xB1
+\xF5\x88\xEF)\x8Abii\xE9ĉ?\xFF\xF9\xCFϝ;\xE7&\xCD\xBC\xD3\xE2\xE2⫯\xBEz\xF3\xE6\xCDF\xA3\xF1\x95\xAF|ebb–,\xB6\x9B\xBC\x82Uŵk\xD7^|\xF1\xC5_\xFC\xE2o\xBE\xF9f\xA5R\xE9\xD0d\xDC\xD7\xC6\xC6\xC6}\xF4џ\xFC\xE4'\xDF\xFF\xFE\xF7\xF7\xEEݫ\xB1\xD8Vv\xFE\xF4\xA7?}\xFF_\xDDj\xB5\xE6\xE7\xE7\x8F;\xF6\xB3\x9F\xFD\xEC\xFA\xF5\xEB\xB7n\xDD\xEA\xD8`\xDC\xDF\xEA\xF5z\xB3ٜ\x9D\x9D{\xE4\x91G\xC6\xC6\xC64\xDB\xC7\xAC\xA2(\xE6\xE7\xE7\xF9\xCB_>|x\xF3\xEE7\x9D\x80\xFB\xDE[o\xBDU\xADVϞ=[*\x95{\xEC\xB1|\xD0~,\xB6\x89\xF7\xFBb\xA2\xDDn/--\xBD\xF4\xD2K/\xBC\xF0\xC2իW\xEB\xF5zG\xC7\xA07\xD4\xEB\xF5\xABW\xAF\xBE\xF0\xC2/\xBD\xF4\xD2\xD2\xD2R'n\x80\xF7\xA0\xF7X\x9B\xEF<~\xFC\xF8\x91#G\xD4\xC8fc9r\xE4\xF8\xF1㫫\xAB\x8B\xED\xE0}V\xB5Z\x9D\x99\x999|\xF8\xF0ŋ\xD5T\xBD^\xBFx\xF1\xE2\xE1Çgff\xAA\xD5j\xB7ǁ\x8E{\xEF\xC0j4gϞ}\xFE\xF9\xE7ggg\xDD\x80g}}}vv\xF6\xF9\xE7\x9F?{\xF6l\xA3\xD1\xE8\xF68\xD0Y\xEFqV\xBB\xDD^\\<v\xEC\xD8\xCC\xCC\xCC\xC6\xC6\xC6ݙ	\x80\x9E\xB4\xB1\xB1133\xF3\xF9\xCF\xFE\x91G\x99\x9A\x9A\xB2\xE1\x9D\xF6+X\x95Jeff\xE6رc\xB5Z\xED\xEE@\xAB\xD5j\x9B/\xDA\xA3Ho\xFB_\x81\xD5l6\xE7\xE6\xE6\x8E=z\xF9\xF2e˹|t\x8DF\xE3\xF2\xE5\xCBG\x8F\x9D\x9B\x9Bk6\x9B\xDD:\xE5\xD6\xCA\xCA\xCA\xF4\xF4\xF4ɓ'ml \xA5^\xAF\x9F<yrzzڝ\xD6\xE8a\xEFX\xCDf\xF3\xE2ŋ'N\x9C\xB8}\xFB\xF6\xDD\x80\x9Ew\xFB\xF6\xED'N\\xBAt\xA9\xD5ju{\xE8\x88w
+\xAC۷o\xFF\xF9\xCF~\xFD\xF5׋\xA2\xB8\x9B\xD0\xF3\x8A\xA28{\xF6쫯\xBEj'\xBDj\xEB\xC0*\x8A\xE2\xEAի\xFAӟ\xDC\x80N\xA8T*\xD3\xD3\xD3W\xAF^\xF52\x9E\x9E\xB4u`\xAD\xAD\xAD\x9D<y\xF2̙3v \xD0	\xCDf\xF3o\xFB\xDBk\xAF\xBD\xE6\x84Ez\xD2ցU\xADVϜ9c\xFB!\x9D\xB3\xB2\xB2r\xEA\xD4)W	\xE9I[V\xA5R\x99\x9B\x9B\xF3\xE6A:\xA7^\xAF\xBF\xF1\xC6\x8B\x9E\xB4E`5\x8D\x85\x85\x85K\x97.\xDD\xF5a\xD8^.]\xBA\xB4\xB0\xB0\xE0\xA8Ez\xCF\x81\xB5\xBE\xBE>77\xB7\xBC\xBC|\xF7\xA7`[Y^^\x9E\x9B\x9B\xB3
+\x8B\xDE\xF3\xAE\x81\xE5\xD7\x80N\xF3\x87^\xB5\xF5%\xC2\xEBׯ;\xFC
+\x80Nk\xB5Zׯ_w\x89\x90޳E`\xB5\xDB\xEDZ\xAD\xD6n\xB7\xEF\xFE4l+\xFE\xE3Ы\xB6^\xC1Z]]u\xF2\x9DV\xC5\xEA\xEA\xAA,z\xCF\x81\xD5j\xB5\xBCi\x80\xBB\xA3R\xA9ؔB\xEF\xE9\xFF\xEF
+<x\xB0\xD9l:\xC6\x80\x8E\xEA\xEF\xEF?x\xF0\xE0\xF0\xF0p\xB7\x81\xB0\xBE\xFF\xBE\xF2\xDDh4\xABժ\xAB\x84tT\xA9Tٳg\xCF\xC0\xC0@\xB7g\x81\xA4-\x80\x8Fb\xEB[\xE5\xF0\xA1	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa L`\x84	,\x800\x81&\xB0\xC2@\x98\xC0Xa \xEC\xFF\xC6/\xE0$/IEND\xAEB`\x82
\ No newline at end of file

Modified: trunk/Source/WebCore/ChangeLog (139255 => 139256)


--- trunk/Source/WebCore/ChangeLog	2013-01-10 00:39:55 UTC (rev 139255)
+++ trunk/Source/WebCore/ChangeLog	2013-01-10 00:44:57 UTC (rev 139256)
@@ -1,3 +1,21 @@
+2013-01-09  Takashi Sakamoto  <ta...@google.com>
+
+        border-radius with box-shadow is not rendered correctly
+        https://bugs.webkit.org/show_bug.cgi?id=106404
+
+        Reviewed by Hajime Morita.
+
+        RoundedRect::isRenderable is wrong. So RoundedRect::adjustRadii
+        is invoked for renderable RoundedRects.
+
+        Test: fast/borders/border-radius-with-box-shadow.html
+
+        * platform/graphics/RoundedRect.cpp:
+        (WebCore::RoundedRect::isRenderable):
+        Have to compare topLeft's height plus bottomLeft's height with
+        rect's height and to compare topRight's height plus bottomRight's
+        height with rect's height.
+
 2013-01-09  Joanmarie Diggs  <jdi...@igalia.com>
 
         [GTK] accessibility/aria-labelledby-overrides-label.html requires a proper baseline

Modified: trunk/Source/WebCore/platform/graphics/RoundedRect.cpp (139255 => 139256)


--- trunk/Source/WebCore/platform/graphics/RoundedRect.cpp	2013-01-10 00:39:55 UTC (rev 139255)
+++ trunk/Source/WebCore/platform/graphics/RoundedRect.cpp	2013-01-10 00:44:57 UTC (rev 139256)
@@ -158,8 +158,8 @@
 {
     return m_radii.topLeft().width() + m_radii.topRight().width() <= m_rect.width()
         && m_radii.bottomLeft().width() + m_radii.bottomRight().width() <= m_rect.width()
-        && m_radii.topLeft().height() + m_radii.topRight().height() <= m_rect.height()
-        && m_radii.bottomLeft().height() + m_radii.bottomRight().height() <= m_rect.height();
+        && m_radii.topLeft().height() + m_radii.bottomLeft().height() <= m_rect.height()
+        && m_radii.topRight().height() + m_radii.bottomRight().height() <= m_rect.height();
 }
 
 void RoundedRect::adjustRadii()

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to