Move use_webview_internal_framework to android_webview/BUILD.gn

build/config/android/config.gni

Index: build/config/android/config.gni
diff --git a/build/config/android/config.gni b/build/config/android/config.gni
index 2b5bb7117f21b57271635388321a6d566ad53170..14857a38ffbde7cfdde6c1de9e8441a06a771e3d 100644
--- a/build/config/android/config.gni
+++ b/build/config/android/config.gni
@@ -108,9 +108,6 @@ if (is_android) {
     # Adds intrumentation to each function. Writes a file with the order that
     # functions are called at startup.
     use_order_profiling = false
-
-    # Use the internal version of the framework to build Android WebView.
-    use_webview_internal_framework = false

[michaelbai, 2016/01/06 03:53:42]

Sorry, I still don't understand what caused the error and why this variable need
to be removed.

From gn help declare_args, 

"Introduces the given arguments into the current scope. If they are
  not specified on the command line or in a toolchain's arguments,
  the default values given in the declare_args block will be used."

IIUC, the default value of use_webview_internal_framework should be defined
here, then, it could be used every where, did I miss something?

[agrieve, 2016/01/06 04:16:04]

Looks like the problem doesn't happen anymore because
8d2e9c0cb301a83e5e700166e3119ca66bb88b75 was reverted ("Port Monochrome to GN").
Maybe this was the reason for the revert?

If you revert the revert, I suspect you'll see the gn gen failure. 

The problem is just that the order is:

1. GN starts parsing //build/android/config.gni
2. GN hits import("//clank/config.gni")
3. //clank/config.gni references use_webview_internal_framework
4. GN hits the declare_args() block which defines  
use_webview_internal_framework

#3 fails because it needs to happen after #4

   }
 
   # Host stuff -----------------------------------------------------------------