Annotated functions/classes more cleanly kept

chrome/android/webapk/shell_apk/BUILD.gn

 # Copyright 2015 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.
 
 import("//build/config/android/rules.gni")
 
 declare_args() {
   # The origin URL of the WebAPK. Used to generate a unique package name for
   # WebAPK. Example: "foo.com"
   webapk_manifest_package_origin = "template"
@@ skipping 68 matching lines @@
     "src/org/chromium/webapk/shell_apk/WebApkSandboxedProcessService2.java",
     "src/org/chromium/webapk/shell_apk/WebApkServiceFactory.java",
   ]
   deps = [
     ":dex_loader_java",
     ":shell_apk_resources",
     "//chrome/android/webapk/libs/common:common_java",
   ]
   if (!is_java_debug) {
     proguard_enabled = true
-    proguard_configs = [ "//chrome/android/java/proguard.flags" ]
+    proguard_configs = [
+      "//chrome/android/java/proguard.flags",
+      "//base/base_proguard_config.flags",

[Yaron, 2016/07/18 13:41:21]

This is a bit awkward as webapk shell doesn't depend on base. But the proguard
definitions there aren't strictly about "base" either.

[agrieve, 2016/07/18 15:53:16]

//build/android would be another place that makes sense if we wanted to split
them up even further (into base & non-base things)

+    ]
   }
 }
 
 android_library("shell_apk_javatests") {
   testonly = true
   java_files =
       [ "javatests/src/org/chromium/webapk/shell_apk/DexLoaderTest.java" ]
   deps = [
     ":dex_loader_java",
     "//base:base_java",
     "//base:base_java_test_support",
     "//chrome/android/webapk/libs/common:common_java",
     "//content/public/test/android:content_java_test_support",
   ]
   srcjar_deps = [ "javatests/dex_optimizer:dex_optimizer_service_aidl" ]
 }