Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode exception

base/android/build_info.h

Index: base/android/build_info.h
diff --git a/base/android/build_info.h b/base/android/build_info.h
index 3c4c86c1d7cefe8d659ec0823fd73a9ecbd24186..acb6bfaeeb4e724d77c8e9e5bd60fe9532fc81df 100644
--- a/base/android/build_info.h
+++ b/base/android/build_info.h
@@ -8,6 +8,7 @@
 #include <jni.h>
 
 #include <string>
+#include <vector>
 
 #include "base/base_export.h"
 #include "base/macros.h"
@@ -117,27 +118,27 @@ class BASE_EXPORT BuildInfo {
  private:
   friend struct BuildInfoSingletonTraits;
 
-  explicit BuildInfo(JNIEnv* env);
+  explicit BuildInfo(const std::vector<std::string>& params);
 
   // Const char* is used instead of std::strings because these values must be
   // available even if the process is in a crash state. Sadly
   // std::string.c_str() doesn't guarantee that memory won't be allocated when
   // it is called.
+  const char* const brand_;
   const char* const device_;
+  const char* const android_build_id_;
   const char* const manufacturer_;
   const char* const model_;
-  const char* const brand_;
-  const char* const android_build_id_;
-  const char* const android_build_fp_;
-  const char* const gms_version_code_;
-  const char* const package_version_code_;
-  const char* const package_version_name_;
+  const int sdk_int_;
+  const char* const build_type_;
   const char* const package_label_;
   const char* const package_name_;
-  const char* const build_type_;
+  const char* const package_version_code_;
+  const char* const package_version_name_;
+  const char* const android_build_fp_;
+  const char* const gms_version_code_;
   // Not needed by breakpad.
   const std::string extracted_file_suffix_;
-  const int sdk_int_;
   // This is set via set_java_exception_info, not at constructor time.
   const char* java_exception_info_;