Build libmonochrome.so with secondary toolchain

build/config/android/config.gni

 # Copyright 2014 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.
 
 # This file contains common system config stuff for the Android build.
 
+import("//build/config/android/secondary_abi.gni")

[agrieve, 2016/07/20 02:11:03]

Having this as a separate file doesn't really make sense to me if it's just
going to be imported by the main config.gni. I think either:
1. Don't have it imported here, or
2. Inline into the logic below.

[michaelbai, 2016/07/20 16:51:10]

It was used in more than one place, moved into this file. 

+
 if (is_android) {
   assert(rebase_path("//", root_build_dir) == "../../",
          "Android output directory must be nested 2 levels within src/ (" +
              "e.g.: out-gn/Debug). http://crbug.com/412935")
 
   import("//build_overrides/build.gni")
 
   has_chrome_android_internal =
       exec_script("//build/dir_exists.py",
                   [ rebase_path("//clank", root_build_dir) ],
@@ skipping 242 matching lines @@
   android_objcopy = "${android_tool_prefix}objcopy"
   android_gdbserver =
       "$android_ndk_root/prebuilt/$android_prebuilt_arch/gdbserver/gdbserver"
 
   # Toolchain stuff ------------------------------------------------------------
 
   android_libcpp_root = "$android_ndk_root/sources/cxx-stl/llvm-libc++"
 
   # ABI ------------------------------------------------------------------------
 
-  # Intentionally do not define android_app_secondary_abi for 64-bit
+  # Intentionally do not define android_app_secondary_abi for 32-bit
   # android_app_abi, since they are not used.
   if (current_cpu == "x86") {
     android_app_abi = "x86"
-    android_app_secondary_abi = "x86_64"
   } else if (current_cpu == "arm") {
     import("//build/config/arm.gni")
     if (arm_version < 7) {
       android_app_abi = "armeabi"
     } else {
       android_app_abi = "armeabi-v7a"
     }
-    android_app_secondary_abi = "arm64-v8a"
   } else if (current_cpu == "mipsel") {
     android_app_abi = "mips"
-    android_app_secondary_abi = "mips64"
   } else if (current_cpu == "x64") {
     android_app_abi = "x86_64"
+    android_app_secondary_abi = "x86"

[agrieve, 2016/07/20 02:11:03]

If we're going to have a separate file for secondary_abi things, this value
should probably be defined there.

   } else if (current_cpu == "arm64") {
     android_app_abi = "arm64-v8a"
+    android_app_secondary_abi = "armeabi-v7a"
   } else if (current_cpu == "mips64el") {
     android_app_abi = "mips64"
+    android_app_secondary_abi = "mips"
   } else {
     assert(false, "Unknown Android ABI: " + current_cpu)
   }
 
   if (android_libcpp_lib_dir == "") {
     android_libcpp_lib_dir = "${android_libcpp_root}/libs/${android_app_abi}"
   }
 }