Title: [172218] trunk
Revision
172218
Author
[email protected]
Date
2014-08-07 11:23:13 -0700 (Thu, 07 Aug 2014)

Log Message

border-radius on html does not render properly.
https://bugs.webkit.org/show_bug.cgi?id=135706

Reviewed by Simon Fraser.

Ensure that background is initialized when border-radius is present.

Source/WebCore:

Test: fast/borders/border-radius-on-html.html

* rendering/RenderView.cpp:
(WebCore::rendererObscuresBackground):

LayoutTests:

* fast/borders/border-radius-on-html-expected.html: Added.
* fast/borders/border-radius-on-html.html: Added.

Modified Paths

Added Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (172217 => 172218)


--- trunk/LayoutTests/ChangeLog	2014-08-07 17:57:51 UTC (rev 172217)
+++ trunk/LayoutTests/ChangeLog	2014-08-07 18:23:13 UTC (rev 172218)
@@ -1,3 +1,15 @@
+2014-08-07  Zalan Bujtas  <[email protected]>
+
+        border-radius on html does not render properly.
+        https://bugs.webkit.org/show_bug.cgi?id=135706
+
+        Reviewed by Simon Fraser.
+
+        Ensure that background is initialized when border-radius is present.
+
+        * fast/borders/border-radius-on-html-expected.html: Added.
+        * fast/borders/border-radius-on-html.html: Added.
+
 2014-08-07  Mark Hahnenberg  <[email protected]>
 
         for-in is failing fast/dom/dataset-xhtml.xhtml and dataset.html tests

Added: trunk/LayoutTests/fast/borders/border-radius-on-html-expected.html (0 => 172218)


--- trunk/LayoutTests/fast/borders/border-radius-on-html-expected.html	                        (rev 0)
+++ trunk/LayoutTests/fast/borders/border-radius-on-html-expected.html	2014-08-07 18:23:13 UTC (rev 172218)
@@ -0,0 +1,23 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>This tests that border-radius on html renders correctly.</title>
+<style>
+  body { 
+    margin: 0px;
+  }
+  div {
+    border-radius: 100%;
+    background-color: green;
+  }
+</style>
+</head>
+<body>
+<div id=foo></div>
+<script>
+var container = document.getElementById("foo");
+container.style.width = window.innerWidth  + "px";
+container.style.height = window.innerHeight + "px";
+</script>
+</body>
+</html>

Added: trunk/LayoutTests/fast/borders/border-radius-on-html.html (0 => 172218)


--- trunk/LayoutTests/fast/borders/border-radius-on-html.html	                        (rev 0)
+++ trunk/LayoutTests/fast/borders/border-radius-on-html.html	2014-08-07 18:23:13 UTC (rev 172218)
@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>This tests that border-radius on html renders correctly.</title>
+<style>
+  html {
+    border-radius: 100%;
+    background-color: green;
+  }
+</style>
+</head>
+<body>
+</body>
+</html>

Modified: trunk/Source/WebCore/ChangeLog (172217 => 172218)


--- trunk/Source/WebCore/ChangeLog	2014-08-07 17:57:51 UTC (rev 172217)
+++ trunk/Source/WebCore/ChangeLog	2014-08-07 18:23:13 UTC (rev 172218)
@@ -1,3 +1,17 @@
+2014-08-07  Zalan Bujtas  <[email protected]>
+
+        border-radius on html does not render properly.
+        https://bugs.webkit.org/show_bug.cgi?id=135706
+
+        Reviewed by Simon Fraser.
+
+        Ensure that background is initialized when border-radius is present.
+
+        Test: fast/borders/border-radius-on-html.html
+
+        * rendering/RenderView.cpp:
+        (WebCore::rendererObscuresBackground):
+
 2014-08-07  Pratik Solanki  <[email protected]>
 
         Random resource replacement on beta.icloud.com

Modified: trunk/Source/WebCore/rendering/RenderView.cpp (172217 => 172218)


--- trunk/Source/WebCore/rendering/RenderView.cpp	2014-08-07 17:57:51 UTC (rev 172217)
+++ trunk/Source/WebCore/rendering/RenderView.cpp	2014-08-07 18:23:13 UTC (rev 172218)
@@ -523,6 +523,9 @@
     if (rootObject->rendererForRootBackground().style().backgroundClip() == TextFillBox)
         return false;
 
+    if (style.hasBorderRadius())
+        return false;
+
     return true;
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to