Removed callbacks to JNI functions + added gzip compressed seed support & pulling response time

components/variations/android/variations_seed_bridge.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 "components/variations/android/variations_seed_bridge.h"
 
 #include <jni.h>
 #include <vector>
 
 #include "base/android/jni_android.h"
@@ skipping 21 matching lines @@
 
 namespace variations {
 namespace android {
 
 bool RegisterVariationsSeedBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 void GetVariationsFirstRunSeed(std::string* seed_data,
                                std::string* seed_signature,
-                               std::string* seed_country) {
+                               std::string* seed_country,
+                               base::Time* response_time,
+                               bool* is_gzip_compressed) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jbyteArray> j_seed_data =
       Java_VariationsSeedBridge_getVariationsFirstRunSeedData(
           env, GetApplicationContext());
   ScopedJavaLocalRef<jstring> j_seed_signature =
       Java_VariationsSeedBridge_getVariationsFirstRunSeedSignature(
           env, GetApplicationContext());
   ScopedJavaLocalRef<jstring> j_seed_country =
       Java_VariationsSeedBridge_getVariationsFirstRunSeedCountry(
           env, GetApplicationContext());
+  ScopedJavaLocalRef<jstring> j_response_time =
+      Java_VariationsSeedBridge_getVariationsFirstRunSeedDate(
+          env, GetApplicationContext());
+  jboolean j_is_gzip_compressed =
+      Java_VariationsSeedBridge_getVariationsFirstRunSeedIsGzipCompressed(
+          env, GetApplicationContext());
+  base::Time::FromString(ConvertJavaStringToUTF8(env, j_response_time).c_str(),

[Alexei Svitkine (slow), 2015/11/12 16:26:55]

I think this logic should be in the variations code. This file should just plumb
things without extra logic.

[Alexander Agulenko, 2015/11/12 20:07:42]

Acknowledged (still working on it).

[Alexander Agulenko, 2015/11/12 20:15:40]

Done.

+                         response_time);
   *seed_data = JavaByteArrayToString(env, j_seed_data.obj());
   *seed_signature = ConvertJavaStringToUTF8(j_seed_signature);
   *seed_country = ConvertJavaStringToUTF8(j_seed_country);
+  *is_gzip_compressed = static_cast<bool>(j_is_gzip_compressed);
 }
 
 }  // namespace android
 }  // namespace variations