commit 4c0a170585d5c8a1f3508ac55f799ebaf86e91db
author Mathias Agopian <mathias@google.com> Fri Aug 31 12:45:33 2012 -0700
committer Mathias Agopian <mathias@google.com> Fri Aug 31 18:38:22 2012 -0700
tree a5fb6a7ab7accc4dc0265572507e73c8142740b8
parent 2098517e3e12a401005d7a7510d6c4943707b98d

don't generate a new Region when translating by 0,0

since regions are copy-on-write, this prevents to
duplicate the region entirely and offseting by 0,0 is
fairly common.

Change-Id: I9b8c286315a2e00dda01c2456397d72b5e12006b

libs/ui/Region.cpp

diff --git a/libs/ui/Region.cpp b/libs/ui/Region.cpp
index a3d8b01..2569ae1 100644
--- a/libs/ui/Region.cpp
+++ b/libs/ui/Region.cpp
@@ -529,7 +529,7 @@
 
 void Region::translate(Region& reg, int dx, int dy)
 {
-    if (!reg.isEmpty()) {
+    if ((dx || dy) && !reg.isEmpty()) {
 #if VALIDATE_REGIONS
         validate(reg, "translate (before)");
 #endif