Reland: Added a debug info UI for Blimp

blimp/client/app/android/blimp_client_session_android.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "blimp/client/app/android/blimp_client_session_android.h"
 
 #include <string>
 
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
@@ skipping 96 matching lines @@
       assignment.engine_endpoint.address(), assignment.engine_endpoint.port());
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_BlimpClientSession_onAssignmentReceived(
       env, java_obj_.obj(), static_cast<jint>(result),
       base::android::ConvertUTF8ToJavaString(env, engine_ip).obj(),
       base::android::ConvertUTF8ToJavaString(env, blimp::kEngineVersion).obj());
 
   BlimpClientSession::OnAssignmentConnectionAttempted(result, assignment);
 }
 
+base::android::ScopedJavaLocalRef<jintArray>
+BlimpClientSessionAndroid::GetDebugInfo(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& jobj) {
+  BlimpConnectionStatistics* stats =
+      BlimpClientSession::GetBlimpConnectionStatistics();
+  int metrics[] = {stats->Get(BlimpConnectionStatistics::BYTES_RECEIVED),
+                   stats->Get(BlimpConnectionStatistics::BYTES_SENT),
+                   stats->Get(BlimpConnectionStatistics::COMMIT)};

[Wez, 2016/06/07 00:30:12]

As noted in the implementation, you're creating three separate histogram
snapshots to query from - IIUC all you actually want to do is have
GetBlimpConnectionStatistics() return a snapshot that you can query instead.

+  return base::android::ToJavaIntArray(env, metrics, arraysize(metrics));
+}
+
 }  // namespace client
 }  // namespace blimp