Implement OpenGL backend in Android viewer app

tools/viewer/Viewer.cpp

Index: tools/viewer/Viewer.cpp
diff --git a/tools/viewer/Viewer.cpp b/tools/viewer/Viewer.cpp
index 9f3b8f2e27df62d505737ac37a8d0503c7d63d98..4dc87226354bab741d07350522dd1ea6a377feb9 100644
--- a/tools/viewer/Viewer.cpp
+++ b/tools/viewer/Viewer.cpp
@@ -422,9 +422,11 @@ void Viewer::updateUIState() {
     // We will be able to change the backend too.
     Json::Value backendState(Json::objectValue);
     backendState[kName] = kBackendStateName;
-    backendState[kValue] = fBackendType == sk_app::Window::kVulkan_BackendType ?
-            "Vulkan" : "Other than Vulkan";
+    backendState[kValue] = kBackendTypeStrings[fBackendType];
     backendState[kOptions] = Json::Value(Json::arrayValue);
+    for(auto str : kBackendTypeStrings) {
+        backendState[kOptions].append(Json::Value(str));
+    }
 
     Json::Value state(Json::arrayValue);
     state.append(slideState);
@@ -434,7 +436,8 @@ void Viewer::updateUIState() {
 }
 
 void Viewer::onUIStateChanged(const SkString& stateName, const SkString& stateValue) {
-    // Currently, we only recognize the Slide state
+    // After the change, please call updateUIState no notify the frontend (e.g., Android app).

[scroggo, 2016/06/02 18:25:16]

to* notify?

Also, who is this comment for? The caller of this method? This seems like the
wrong place for it, if so.

[liyuqian, 2016/06/02 18:47:13]

This comment is for those who will add more features to onUIStateChanged. For
example, when I add the feature of handling backend state change, I should call
updateUIState (line 464) to tell the frontend to update its UI to reflect the
backend change. I'll update the comment to clarify the audience of this comment.

+    // For slide change, updateUIState is called inside setupCurrentSlide.
     if (stateName.equals(kSlideStateName)) {
         int previousSlide = fCurrentSlide;
         fCurrentSlide = 0;
@@ -449,6 +452,20 @@ void Viewer::onUIStateChanged(const SkString& stateName, const SkString& stateVa
             fCurrentSlide = previousSlide;
             SkDebugf("Slide not found: %s", stateValue.c_str());
         }
+    } else if (stateName.equals(kBackendStateName)) {
+        for(int i=0; i<sk_app::Window::kBackendTypeCount; i++) {

[scroggo, 2016/06/02 18:25:16]

nit: Typically we put spaces around operators:

for (int i = 0; i < ...; i++) {

(also space after "for")

[liyuqian, 2016/06/02 18:47:13]

Done.

+            if (stateValue.equals(kBackendTypeStrings[i])) {
+                if (fBackendType != i) {
+                    fBackendType = (sk_app::Window::BackendType)i;
+                    fWindow->detach();
+                    fWindow->attach(fBackendType, DisplayParams());
+                    fWindow->inval();
+                    updateTitle();
+                    updateUIState();
+                }
+                break;
+            }
+        }
     } else {
         SkDebugf("Unknown stateName: %s", stateName.c_str());
     }