Title: [94698] trunk/Source/WebCore
Revision
94698
Author
[email protected]
Date
2011-09-07 12:27:36 -0700 (Wed, 07 Sep 2011)

Log Message

FELighting is using width instead of height to determine skip for parallel(n>2) case
https://bugs.webkit.org/show_bug.cgi?id=67719
<rdar://problem/10086178>

Reviewed by Dirk Schulze.

No new tests, because the bug only exhibits under conditions which are currently
impossible by default (using more than 2 cores) on some platforms.

* platform/graphics/filters/FELighting.cpp:
(WebCore::FELighting::platformApplyGeneric):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (94697 => 94698)


--- trunk/Source/WebCore/ChangeLog	2011-09-07 19:20:13 UTC (rev 94697)
+++ trunk/Source/WebCore/ChangeLog	2011-09-07 19:27:36 UTC (rev 94698)
@@ -1,3 +1,17 @@
+2011-09-07  Tim Horton  <[email protected]>
+
+        FELighting is using width instead of height to determine skip for parallel(n>2) case
+        https://bugs.webkit.org/show_bug.cgi?id=67719
+        <rdar://problem/10086178>
+
+        Reviewed by Dirk Schulze.
+
+        No new tests, because the bug only exhibits under conditions which are currently
+        impossible by default (using more than 2 cores) on some platforms.
+
+        * platform/graphics/filters/FELighting.cpp:
+        (WebCore::FELighting::platformApplyGeneric):
+
 2011-09-07  W. James MacLean  <[email protected]>
 
         Eliminate motion jitter in animated, blurred SVG image

Modified: trunk/Source/WebCore/platform/graphics/filters/FELighting.cpp (94697 => 94698)


--- trunk/Source/WebCore/platform/graphics/filters/FELighting.cpp	2011-09-07 19:20:13 UTC (rev 94697)
+++ trunk/Source/WebCore/platform/graphics/filters/FELighting.cpp	2011-09-07 19:27:36 UTC (rev 94698)
@@ -261,7 +261,7 @@
         int job = parallelJobs.numberOfJobs();
         if (job > 1) {
             int yStart = 1;
-            int yStep = (data.widthDecreasedByOne - 1) / job;
+            int yStep = (data.heightDecreasedByOne - 1) / job;
             for (--job; job >= 0; --job) {
                 PlatformApplyGenericParameters& params = parallelJobs.parameter(job);
                 params.filter = this;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to