Move inval dedup to Window for wider usages.

tools/viewer/sk_app/Window.h

 /*
 * Copyright 2016 Google Inc.
 *
 * Use of this source code is governed by a BSD-style license that can be
 * found in the LICENSE file.
 */
 
 #ifndef Window_DEFINED
 #define Window_DEFINED
 
 #include "DisplayParams.h"
 #include "SkRect.h"
 #include "SkTouchGesture.h"
 #include "SkTypes.h"
 
 class SkCanvas;
 
 namespace sk_app {
 
 class WindowContext;
 
 class Window {
 public:
     static Window* CreateNativeWindow(void* platformData);
 
     virtual ~Window() {};
 
     virtual void setTitle(const char*) = 0;
     virtual void show() = 0;
-    virtual void inval() = 0;
+
+    // Check if the current window has any pending inval; call inval if not.

[djsollen, 2016/05/23 17:06:48]

update the comment...

// shedules an invalidation event for window if one is not currently pending.

+    void inval();
 
     virtual bool scaleContentToFit() const { return false; }
     virtual bool supportsContentRect() const { return false; }
     virtual SkRect getContentRect() { return SkRect::MakeEmpty(); }
 
     enum BackendType {
         kNativeGL_BackendType,
         kVulkan_BackendType,
 
         kLast_BackendType = kVulkan_BackendType
@@ skipping 116 matching lines @@
     OnKeyFunc    fKeyFunc;
     void*        fKeyUserData;
     OnMouseFunc  fMouseFunc;
     void*        fMouseUserData;
     OnTouchFunc  fTouchFunc;
     void*        fTouchUserData;
     OnPaintFunc  fPaintFunc;
     void*        fPaintUserData;
 
     WindowContext* fWindowContext = nullptr;
+
+    // Make sure that either onPaint or markInvalReceived is called in the subclasses'

[djsollen, 2016/05/23 17:06:48]

I think this comment is outdated.  the client window doesn't need to worry about
this unless they don't call onPaint as part of handling the inval event.

[liyuqian, 2016/05/23 17:29:51]

In case that the client window doesn't call onPaint (e.g., Android), this
comment reminds it to call markInvalProcessed. I'll update the comment to
reflect the name change of inval to onInval.

+    // implementations of inval. They unset fIsContentInvalided which allow invalIfNeeded
+    // to invalidate the content in the future.
+    virtual void onInval() = 0;
+
+    // Uncheck fIsContentInvalided to allow future inval/onInval.
+    void markInvalProcessed();
+
+    bool fIsContentInvalidated = false;  // use this to avoid duplicate invalidate events
 };
 
 }   // namespace sk_app
 #endif