Added Version Info page to toolbar menu

blimp/client/app/android/blimp_client_session_android.h

Index: blimp/client/app/android/blimp_client_session_android.h
diff --git a/blimp/client/app/android/blimp_client_session_android.h b/blimp/client/app/android/blimp_client_session_android.h
index 681655e7a6a8f1224fc092a3611c5c2e2a2de328..c8b6c511d53ff527aa69f2c789157dc704bdd0e0 100644
--- a/blimp/client/app/android/blimp_client_session_android.h
+++ b/blimp/client/app/android/blimp_client_session_android.h
@@ -31,6 +31,13 @@ class BlimpClientSessionAndroid : public BlimpClientSession {
 
   void Destroy(JNIEnv* env, const base::android::JavaParamRef<jobject>& jobj);
 
+  base::android::ScopedJavaLocalRef<jstring> GetEngineIPAddress(

[Kevin M, 2016/03/31 19:41:48]

Do we need to break these out into separate fields? I think it might be
advantageous to use a single formatted string instead (e.g. GetEngineInfo()). It
could return a value like "ip:127.0.0.1 version:v51.0.123123" That way, if we
decide to add additional information like as locale info or production zone, we
don't have to poke more holes into JNI.

I can see how having separate fields might be nice for analytics KV pairs, but
if we're just driving a dialog box, then one string seems fine?

[shaktisahu, 2016/04/06 06:50:25]

Acknowledged.

+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& jobj) const;
+  base::android::ScopedJavaLocalRef<jstring> GetEngineVersion(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& jobj) const;
+
  private:
   ~BlimpClientSessionAndroid() override;