Check library version and handle library load errors

chrome/app/android/chrome_android_initializer.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 "chrome/app/android/chrome_android_initializer.h"
 
 #include "base/android/jni_android.h"
 #include "base/logging.h"
 #include "chrome/app/android/chrome_main_delegate_android.h"
+#include "chrome/common/chrome_version_info.h"
 #include "content/public/app/android_library_loader_hooks.h"
 #include "content/public/app/content_main.h"
 #include "net/proxy/proxy_resolver_v8.h"
 
 jint RunChrome(JavaVM* vm, ChromeMainDelegateAndroid* main_delegate) {
   base::android::InitVM(vm);
   JNIEnv* env = base::android::AttachCurrentThread();
   if (!content::RegisterLibraryLoaderEntryHook(env))
     return -1;
 
+  // Pass the library version number to content so that we can check it from the
+  // Java side before continuing initialization
+  chrome::VersionInfo vi;
+  content::SetVersionName(vi.Version());

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

Version() returns the version "number" so bit confusing to pass it to set
version "name".

(maybe a comment on content::SetVersionName explaining exactly what the string
set is for, and how it is used, would make sense)

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

I thought originally that this was going to match the Android "VersionName", but
in the end this didn't work. I will change this to Version or VersionNumber (or
VERSION_NUMBER etc.) throughout (even if "1.2.3.4" isn't really a number).
Isn't this covered by the comment in android_library_loader_hooks.h?

+
   DCHECK(main_delegate);
   content::SetContentMainDelegate(main_delegate);
 
   // http://crbug.com/173648 . V8's default isolate is used by ProxyResolverV8
   // to resolve PAC urls. The default isolate is created by static initializer
   // on the shared library thread, and can only be looked up in that thread's
   // TLS.
   net::ProxyResolverV8::RememberDefaultIsolate();
 
   return JNI_VERSION_1_4;
 }