Allow animation in Android viewer

tools/viewer/sk_app/android/surface_glue_android.cpp

 /*
 * Copyright 2016 Google Inc.
 *
 * Use of this source code is governed by a BSD-style license that can be
 * found in the LICENSE file.
 */
 
 #include "surface_glue_android.h"
 
 #include <jni.h>
 #include <pthread.h>
 #include <stdio.h>
 #include <unistd.h>
 #include <unordered_map>
 
 #include <android/input.h>
 #include <android/keycodes.h>
 #include <android/looper.h>
 #include <android/native_window_jni.h>
 
 #include "../Application.h"
 #include "SkTypes.h"
 #include "SkUtils.h"
 #include "Window_android.h"
+#include "SkTime.h"
 
 namespace sk_app {
 
 static const int LOOPER_ID_MESSAGEPIPE = 1;
 
 static const std::unordered_map<int, Window::Key> ANDROID_TO_WINDOW_KEYMAP({
     {AKEYCODE_SOFT_LEFT, Window::Key::kLeft},
     {AKEYCODE_SOFT_RIGHT, Window::Key::kRight}
 });
 
@@ skipping 118 matching lines @@
             break;
         }
         default: {
             // do nothing
         }
     }
 
     return 1;  // continue receiving callbacks
 }
 
+static double now_ms() { return SkTime::GetMSecs(); }
+
 void* SkiaAndroidApp::pthread_main(void* arg) {
     SkDebugf("pthread_main begins");
 
     auto skiaAndroidApp = (SkiaAndroidApp*)arg;
 
     // Because JNIEnv is thread sensitive, we need AttachCurrentThread to set our fPThreadEnv
     skiaAndroidApp->fJavaVM->AttachCurrentThread(&(skiaAndroidApp->fPThreadEnv), nullptr);
 
     ALooper* looper = ALooper_prepare(ALOOPER_PREPARE_ALLOW_NON_CALLBACKS);
     pipe(skiaAndroidApp->fPipes);
     ALooper_addFd(looper, skiaAndroidApp->fPipes[0], LOOPER_ID_MESSAGEPIPE, ALOOPER_EVENT_INPUT,
                   message_callback, skiaAndroidApp);
 
-    int ident;
-    int events;
-    struct android_poll_source* source;
-
     skiaAndroidApp->fApp = Application::Create(0, nullptr, skiaAndroidApp);
 
-    while ((ident = ALooper_pollAll(-1, nullptr, &events, (void**)&source)) >= 0) {
-        SkDebugf("ALooper_pollAll ident=%d", ident);
+    double currentTime = 0.0;
+    double previousTime = 0.0;
+    while (true) {
+        const int ident = ALooper_pollAll(0, nullptr, nullptr, nullptr);

[jvanverth1, 2016/06/06 14:18:11]

I'm surprised a timeout of 0 doesn't cause a busy loop (i.e., eat up 100% of
CPU). On the other hand, a timeout of 1 seems like it would affect our timing.

[liyuqian, 2016/06/06 14:30:46]

I checked the CPU usage on Android and it indeed didn't eat up 100% of CPU.

+
+        if (ident >= 0) {
+            SkDebugf("Unhandled ALooper_pollAll ident=%d !", ident);
+        } else {
+            previousTime = currentTime;
+            currentTime = now_ms();
+            skiaAndroidApp->fApp->onIdle(currentTime - previousTime);
+        }
     }
 
+    SkDebugf("pthread_main ends");
+
     return nullptr;
 }
 
 extern "C"  // extern "C" is needed for JNI (although the method itself is in C++)
     JNIEXPORT jlong JNICALL
     Java_org_skia_viewer_ViewerApplication_createNativeApp(JNIEnv* env, jobject application) {
     SkiaAndroidApp* skiaAndroidApp = new SkiaAndroidApp(env, application);
     return (jlong)((size_t)skiaAndroidApp);
 }
 
@@ skipping 53 matching lines @@
     const char* nameChars = env->GetStringUTFChars(stateName, nullptr);
     const char* valueChars = env->GetStringUTFChars(stateValue, nullptr);
     message.stateName = new SkString(nameChars);
     message.stateValue = new SkString(valueChars);
     skiaAndroidApp->postMessage(message);
     env->ReleaseStringUTFChars(stateName, nameChars);
     env->ReleaseStringUTFChars(stateValue, valueChars);
 }
 
 }  // namespace sk_app