Skip to content

Commit

Permalink
Remove android_support_annotations target.
Browse files Browse the repository at this point in the history
android-support-annotations.jar and android-support-v13.jar both define android.support.annotations. Since we use both, this can produce errors building when we try to use android.support.annotations in a new target (eg com.android.dex.DexException: Multiple dex files define Landroid/support/annotation/AnimRes).

android-support-annotations.jar contains a subset of android-support-v13.jar, so I am removing it and updating all dependencies to point to the latter.

BUG=623012

Review-Url: https://codereview.chromium.org/2137433002
Cr-Commit-Position: refs/heads/master@{#404371}
  • Loading branch information
peconn authored and Commit bot committed Jul 8, 2016
1 parent 1a552c0 commit dd8383a
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 11 deletions.
4 changes: 0 additions & 4 deletions build/secondary/third_party/android_tools/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,6 @@ android_java_prebuilt("android_support_design_java") {
jar_path = "$android_sdk_root/extras/android/support/design/libs/android-support-design.jar"
}

android_java_prebuilt("android_support_annotations_javalib") {
jar_path = "$android_sdk_root/extras/android/support/annotations/android-support-annotations.jar"
}

java_prebuilt("android_support_multidex_java") {
supports_android = true
jar_path = "$android_sdk_root/extras/android/support/multidex/library/libs/android-support-multidex.jar"
Expand Down
4 changes: 2 additions & 2 deletions components/cronet.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@
'cronet_version',
'load_states_list',
'network_quality_observation_source_java',
'../third_party/android_tools/android_tools.gyp:android_support_annotations_javalib',
'../third_party/android_tools/android_tools.gyp:android_support_v13_java',
],
'variables': {
'java_in_dir': 'cronet/android/api',
Expand All @@ -269,7 +269,7 @@
'libcronet',
'net_request_priority_java',
'network_quality_observation_source_java',
'../third_party/android_tools/android_tools.gyp:android_support_annotations_javalib',
'../third_party/android_tools/android_tools.gyp:android_support_v13_java',
],
'variables': {
'java_in_dir': 'cronet/android/java',
Expand Down
4 changes: 2 additions & 2 deletions components/cronet/android/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ android_library("cronet_api") {
]

deps = [
"//third_party/android_tools:android_support_annotations_javalib",
"//third_party/android_tools:android_support_v13_java",
]

srcjar_deps = [
Expand Down Expand Up @@ -313,7 +313,7 @@ android_library("cronet_java") {
":cronet_api",
"//base:base_java",
"//net/android:net_java",
"//third_party/android_tools:android_support_annotations_javalib",
"//third_party/android_tools:android_support_v13_java",
"//third_party/jsr-305:jsr_305_javalib",
]

Expand Down
2 changes: 1 addition & 1 deletion third_party/android_async_task/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ android_library("android_async_task_java") {
java_files = [ "java/src/org/chromium/third_party/android/os/AsyncTask.java" ]
deps = [
"//base:base_java",
"//third_party/android_tools:android_support_annotations_javalib",
"//third_party/android_tools:android_support_v13_java",
]
chromium_code = false
}
2 changes: 1 addition & 1 deletion third_party/android_async_task/README.chromium
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ This contains a copy of Android sdk 23's AsyncTask.java.
Local Modifications:
- Renamed package to org.chromium.third_party.android.os.
- Explicitly import android.os objects.
- Switch to using android.support.annotations.
- Switch to using android.support.annotations (from android.support.v13).
2 changes: 1 addition & 1 deletion third_party/espresso/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ android_java_prebuilt("espresso_contrib_java") {
android_java_prebuilt("espresso_core_java") {
jar_path = "lib/espresso-core-2.2-release-no-dep.jar"
deps = [
"//third_party/android_tools:android_support_annotations_javalib",
"//third_party/android_tools:android_support_v13_java",
"//third_party/guava:guava_java",
"//third_party/hamcrest:hamcrest_core_java",
"//third_party/javax_inject:javax_inject_java",
Expand Down

0 comments on commit dd8383a

Please sign in to comment.