Check library version and handle library load errors

content/app/android/library_loader_hooks.cc

 // Copyright (c) 2012 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 "content/public/app/android_library_loader_hooks.h"
 
 #include "base/android/base_jni_registrar.h"
 #include "base/android/jni_android.h"
 #include "base/android/jni_registrar.h"
 #include "base/android/jni_string.h"
@@ skipping 18 matching lines @@
 #include "net/android/net_jni_registrar.h"
 #include "ui/base/android/ui_jni_registrar.h"
 #include "ui/gfx/android/gfx_jni_registrar.h"
 #include "ui/gl/android/gl_jni_registrar.h"
 #include "ui/shell_dialogs/android/shell_dialogs_jni_registrar.h"
 
 namespace content {
 
 namespace {
 base::AtExitManager* g_at_exit_manager = NULL;
+std::string library_version_name = "";

[joth, 2013/11/11 20:28:56]

g_library_version_name...

[joth, 2013/11/11 20:28:56]

this is a global C++ object, not allowed by style guide.

It'd be nice if you could just make this a const char*
I think that means the caller should just #include chrome_version_info_posix.h
and pass PRODUCT_VERSION as the param.


Tangent: for webview we'd really like chrome_version_info_posix.h (and VERSION
text file) moved into content/ layer so we can use the same version number in
webview UA in future. Seems like doing that would make this CL much simpler too?
(the GetVersinNumber JNI method would just return PRODUCT_VERSION converted to
java string)

[aberent, 2013/11/13 20:20:52]

I have changed it to a const char *, but I am setting it though a strdup from
the version number in chrome_version_info.cc; since otherwise we have to put
lifetime constraints on the argument to SetVersionName, which I never like if I
can avoid it (this sort of constraint is too easy to break in future code
changes). This only happens once per Chrome run, so the performance cost is
trivial.

I can see some dangers in moving the version number down to content, at the
moment it is the Chrome  version number, will this always be in step with the
content version number? Also consider, for example, Opera. They might want to
use this to check that their C++ library is in step with their JAR, and will
probably use their own versioning scheme which will have nothing to do with
Chrome versions; yet they are, as far as I know, using content unchanged.

 }
 
 bool EnsureJniRegistered(JNIEnv* env) {
   static bool g_jni_init_done = false;
 
   if (!g_jni_init_done) {
     if (!base::android::RegisterJni(env))
       return false;
 
     if (!gfx::android::RegisterJni(env))
@@ skipping 90 matching lines @@
   }
 }
 
 bool RegisterLibraryLoaderEntryHook(JNIEnv* env) {
   // We need the AtExitManager to be created at the very beginning.
   g_at_exit_manager = new base::AtExitManager();
 
   return RegisterNativesImpl(env);
 }
 
+void SetVersionName(const std::string & version_name) {

[joth, 2013/11/11 20:28:56]

not space before &

(and in header)

[aberent, 2013/11/13 20:20:52]

Done.

+  library_version_name = version_name;
+}
+
+jstring GetVersionName(JNIEnv* env, jclass clazz) {
+  return env->NewStringUTF(library_version_name.c_str());
+}
+
 }  // namespace content