commit eeb4c387ddd8e52c760b4e4c9fcc6593ccba17be
author bsalomon <bsalomon@google.com> Tue Jul 26 08:50:26 2016 -0700
committer Commit bot <commit-bot@chromium.org> Tue Jul 26 08:50:27 2016 -0700
tree 58e5c6a00fda3d85231e193bdcf75c4700ba1d54
parent 26318c984ffd3ab69fecaa0ba193a3ab6f5c8750

Don't unnecessarily resize windows in sk_app on X
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182163002

Review-Url: https://codereview.chromium.org/2182163002

tools/viewer/sk_app/Window.h

diff --git a/tools/viewer/sk_app/Window.h b/tools/viewer/sk_app/Window.h
index 9ced747..eab8c1d 100644
--- a/tools/viewer/sk_app/Window.h
+++ b/tools/viewer/sk_app/Window.h
@@ -158,8 +158,8 @@
     void onPaint();
     void onResize(uint32_t width, uint32_t height);
 
-    uint32_t width() { return fWidth; }
-    uint32_t height() { return fHeight;  }
+    int width() { return fWidth; }
+    int height() { return fHeight;  }
 
     virtual const DisplayParams& getDisplayParams();
     void setDisplayParams(const DisplayParams& params);
@@ -170,8 +170,8 @@
 protected:
     Window();
 
-    uint32_t     fWidth;
-    uint32_t     fHeight;
+    int          fWidth;
+    int          fHeight;
 
     OnCharFunc   fCharFunc;
     void*        fCharUserData;

tools/viewer/sk_app/unix/Window_unix.h

diff --git a/tools/viewer/sk_app/unix/Window_unix.h b/tools/viewer/sk_app/unix/Window_unix.h
index fb6b22d..b6d40e6 100644
--- a/tools/viewer/sk_app/unix/Window_unix.h
+++ b/tools/viewer/sk_app/unix/Window_unix.h
@@ -63,10 +63,12 @@
         }
     }
 
-    void markPendingResize(int width, int height) { 
-        fPendingWidth = width; 
-        fPendingHeight = height;
-        fPendingResize = true;
+    void markPendingResize(int width, int height) {
+        if (width != fWidth || height != fHeight){
+            fPendingResize = true;
+            fPendingWidth = width;
+            fPendingHeight = height;
+        }
     }
     void finishResize() { 
         if (fPendingResize) {