CC Animation: Make cc/animation an independent GN component.

android_webview/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("//android_webview/system_webview_apk_tmpl.gni")
 import("//android_webview/webview_repack_locales.gni")
 import("//build/config/android/config.gni")
 import("//build/config/android/rules.gni")
 import("//build/config/locales.gni")
 import("//components/spellcheck/spellcheck_build_features.gni")
@@ skipping 455 matching lines @@
     "renderer/print_render_frame_observer.cc",
     "renderer/print_render_frame_observer.h",
   ]
 
   deps = [
     ":generate_aw_resources",
     ":generate_aw_strings",
     ":generate_components_strings",
     "//android_webview/common:version",
     "//android_webview/native:native",
+    "//cc/animation",

[boliu, 2016/11/17 14:50:43]

this is actually not correct I believe

a target should only need to declare its *direct* dependencies. afaict I don't
see any android_webview code including things from cc/animation?

If cc/animation is a dependency of cc/surfaces, then it should be declared a
dependency of cc/surfaces and that will be transitively pulled in here

[loyso (OOO), 2016/11/17 23:45:09]

Done.

     "//cc/surfaces",
     "//components/autofill/content/renderer",
     "//components/cdm/browser",
     "//components/cdm/renderer",
     "//components/crash/content/app",
     "//components/crash/content/browser",
     "//components/metrics",
     "//components/metrics:gpu",
     "//components/metrics:net",
     "//components/metrics:profiler",
@@ skipping 200 matching lines @@
   system_webview_apk_tmpl("system_webview_apk") {
     android_manifest = system_webview_android_manifest
     android_manifest_dep = ":system_webview_manifest"
     deps = [
       ":system_webview_resources",
       "//android_webview/glue",
     ]
     apk_name = "SystemWebView"
   }
 }