Title: [93790] trunk/Source/WebCore
Revision
93790
Author
[email protected]
Date
2011-08-25 08:56:38 -0700 (Thu, 25 Aug 2011)

Log Message

Unreviewed, rolling out r93774.
http://trac.webkit.org/changeset/93774
https://bugs.webkit.org/show_bug.cgi?id=66945

It broke 26 tests on the Qt bot (Requested by ossy on
#webkit).

Patch by Sheriff Bot <[email protected]> on 2011-08-25

* html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::stroke):
* platform/graphics/qt/PathQt.cpp:
(WebCore::Path::boundingRect):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (93789 => 93790)


--- trunk/Source/WebCore/ChangeLog	2011-08-25 15:47:41 UTC (rev 93789)
+++ trunk/Source/WebCore/ChangeLog	2011-08-25 15:56:38 UTC (rev 93790)
@@ -1,3 +1,17 @@
+2011-08-25  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r93774.
+        http://trac.webkit.org/changeset/93774
+        https://bugs.webkit.org/show_bug.cgi?id=66945
+
+        It broke 26 tests on the Qt bot (Requested by ossy on
+        #webkit).
+
+        * html/canvas/CanvasRenderingContext2D.cpp:
+        (WebCore::CanvasRenderingContext2D::stroke):
+        * platform/graphics/qt/PathQt.cpp:
+        (WebCore::Path::boundingRect):
+
 2011-08-25  Andrey Kosyakov  <[email protected]>
 
         Web Inspector: [refactoring] use PopoverHelper to implement popup in the SourceFrame

Modified: trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp (93789 => 93790)


--- trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp	2011-08-25 15:47:41 UTC (rev 93789)
+++ trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp	2011-08-25 15:56:38 UTC (rev 93790)
@@ -924,7 +924,11 @@
         return;
 
     if (!m_path.isEmpty()) {
+#if PLATFORM(QT)
+        FloatRect dirtyRect = m_path.platformPath().controlPointRect();
+#else
         FloatRect dirtyRect = m_path.boundingRect();
+#endif
         // Fast approximation of the stroke's bounding rect.
         // This yields a slightly oversized rect but is very fast
         // compared to Path::strokeBoundingRect().

Modified: trunk/Source/WebCore/platform/graphics/qt/PathQt.cpp (93789 => 93790)


--- trunk/Source/WebCore/platform/graphics/qt/PathQt.cpp	2011-08-25 15:47:41 UTC (rev 93789)
+++ trunk/Source/WebCore/platform/graphics/qt/PathQt.cpp	2011-08-25 15:56:38 UTC (rev 93790)
@@ -1,8 +1,7 @@
 /*
- * Copyright (C) 2006 Zack Rusin <[email protected]>
- * Copyright (C) 2006 Rob Buis <[email protected]>
- * Copyright (C) 2009, 2010 Dirk Schulze <[email protected]>
- * Copyright (C) 2010, 2011 Andreas Kling <[email protected]>
+ * Copyright (C) 2006 Zack Rusin   <[email protected]>
+ *               2006 Rob Buis     <[email protected]>
+ *               2009, 2010 Dirk Schulze <[email protected]>
  *
  * All rights reserved.
  *
@@ -150,7 +149,7 @@
 
 FloatRect Path::boundingRect() const
 {
-    return m_path.controlPointRect();
+    return m_path.boundingRect();
 }
 
 FloatRect Path::strokeBoundingRect(StrokeStyleApplier* applier) const
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to