Title: [199044] trunk/Source/WebCore
Revision
199044
Author
[email protected]
Date
2016-04-04 23:09:01 -0700 (Mon, 04 Apr 2016)

Log Message

[TexMap] resolveOverlaps() should be passed-in the first Region parameter via a reference
https://bugs.webkit.org/show_bug.cgi?id=156158

Reviewed by Antonio Gomes.

* platform/graphics/texmap/TextureMapperLayer.cpp:
(WebCore::resolveOverlaps): Don't copy the Region object that's passed through
the first parameter by accepting a reference to the object instead. This does
modify the passed-in object, but these modifications don't have any effect on
any state via the call sites in TextureMapperLayer::computeOverlapRegions().

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (199043 => 199044)


--- trunk/Source/WebCore/ChangeLog	2016-04-05 05:47:32 UTC (rev 199043)
+++ trunk/Source/WebCore/ChangeLog	2016-04-05 06:09:01 UTC (rev 199044)
@@ -1,3 +1,16 @@
+2016-04-04  Zan Dobersek  <[email protected]>
+
+        [TexMap] resolveOverlaps() should be passed-in the first Region parameter via a reference
+        https://bugs.webkit.org/show_bug.cgi?id=156158
+
+        Reviewed by Antonio Gomes.
+
+        * platform/graphics/texmap/TextureMapperLayer.cpp:
+        (WebCore::resolveOverlaps): Don't copy the Region object that's passed through
+        the first parameter by accepting a reference to the object instead. This does
+        modify the passed-in object, but these modifications don't have any effect on
+        any state via the call sites in TextureMapperLayer::computeOverlapRegions().
+
 2016-04-04  Brady Eidson  <[email protected]>
 
         Modern IDB: Dump blobs to disk before storing them in an object store.

Modified: trunk/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp (199043 => 199044)


--- trunk/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp	2016-04-05 05:47:32 UTC (rev 199043)
+++ trunk/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp	2016-04-05 06:09:01 UTC (rev 199044)
@@ -238,7 +238,7 @@
     m_currentFilters = filters;
 }
 
-static void resolveOverlaps(Region newRegion, Region& overlapRegion, Region& nonOverlapRegion)
+static void resolveOverlaps(Region& newRegion, Region& overlapRegion, Region& nonOverlapRegion)
 {
     Region newOverlapRegion(newRegion);
     newOverlapRegion.intersect(nonOverlapRegion);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to