Upstream DocumentTabModelImpl and related classes

chrome/android/BUILD.gn

Index: chrome/android/BUILD.gn
diff --git a/chrome/android/BUILD.gn b/chrome/android/BUILD.gn
index d4aa47e27880a8bdec560acd2d42f7e8a7874e34..c63bf0d35f627857728a10a02a37e0c0be066222 100644
--- a/chrome/android/BUILD.gn
+++ b/chrome/android/BUILD.gn
@@ -89,6 +89,7 @@ android_library("chrome_java") {
     "//content/public/android:content_java",
     "//printing:printing_java",
     "//sync/android:sync_java",
+    "//third_party/android_protobuf:protobuf_nano_javalib",

[Nico, 2014/12/16 19:52:29]

This doesn't look right to me; the gyp version doesn't need this and
https://code.google.com/p/chromium/codesearch#chromium/src/build/config/andro...
tries to add this automatically where needed from what i understand (?)

[gone, 2014/12/16 19:59:00]

Tried to do it without this line, but PS2 failed the GN bots :/

[cjhopman, 2014/12/16 21:34:00]

In GN, the classpath for a java/android library only contains the things
explicitly listed as dependencies (in gyp it contains all transitive java
dependencies).

The linked template is just for proto_java_library targets.

[cjhopman, 2014/12/16 21:34:00]

Also, GN will probably change to use "all" transitive dependencies for the
classpath since there are some difficult issues with only using direct
dependencies (though this is not necessarily one of those cases).

     "//third_party/android_tools:android_support_v13_java",
     "//third_party/android_tools:android_support_v7_appcompat_java",
     "//third_party/cacheinvalidation:cacheinvalidation_javalib",