Commit 20d743da authored by Andrew Grieve's avatar Andrew Grieve Committed by Commit Bot

Android: Strip some unused mediarouter classes via -assumevalues

Bug: 1000382
Change-Id: I5af4f79b7092c1ad9e459f4f2401e19e08139288
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1797174
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: default avatarSam Maier <smaier@chromium.org>
Commit-Queue: Sam Maier <smaier@chromium.org>
Cr-Commit-Position: refs/heads/master@{#695795}
parent a6d82c7b
...@@ -362,6 +362,28 @@ ...@@ -362,6 +362,28 @@
*** build() return null; *** build() return null;
} }
################################################################################
# ../../third_party/android_deps/support_mediarouter.flags
################################################################################
# Copyright 2019 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.
# Help R8 to remove debug related coded. All of these static constants are
# initialized to: Log.isLoggable(DEBUG, TAG).
-assumevalues class android.support.v7.app.MediaRouteControllerDialogFragment {
static boolean USE_SUPPORT_DYNAMIC_GROUP return false;
}
-assumevalues class android.support.v7.media.MediaRouter {
static boolean DEBUG return false;
}
-assumevalues class android.support.v7.media.RegisteredMediaRouteProvider {
static boolean DEBUG return false;
}
################################################################################ ################################################################################
# ../../third_party/feed/src/src/main/java/com/google/android/libraries/feed/buildtools/proguard/r8_release.flags # ../../third_party/feed/src/src/main/java/com/google/android/libraries/feed/buildtools/proguard/r8_release.flags
################################################################################ ################################################################################
......
...@@ -364,6 +364,9 @@ android_aar_prebuilt("androidx_mediarouter_mediarouter_java") { ...@@ -364,6 +364,9 @@ android_aar_prebuilt("androidx_mediarouter_mediarouter_java") {
":androidx_palette_palette_java", ":androidx_palette_palette_java",
":androidx_recyclerview_recyclerview_java", ":androidx_recyclerview_recyclerview_java",
] ]
# https://crbug.com/1000382
proguard_configs = [ "androidx_mediarouter.flags" ]
} }
# This is generated, do not edit. Update BuildConfigGenerator.groovy instead. # This is generated, do not edit. Update BuildConfigGenerator.groovy instead.
...@@ -618,6 +621,9 @@ android_aar_prebuilt("com_android_support_mediarouter_v7_java") { ...@@ -618,6 +621,9 @@ android_aar_prebuilt("com_android_support_mediarouter_v7_java") {
":com_android_support_recyclerview_v7_java", ":com_android_support_recyclerview_v7_java",
":com_android_support_support_media_compat_java", ":com_android_support_support_media_compat_java",
] ]
# https://crbug.com/1000382
proguard_configs = [ "support_mediarouter.flags" ]
} }
# This is generated, do not edit. Update BuildConfigGenerator.groovy instead. # This is generated, do not edit. Update BuildConfigGenerator.groovy instead.
......
# Copyright 2019 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.
# Help R8 to remove debug related coded. All of these static constants are
# initialized to: Log.isLoggable(DEBUG, TAG).
-assumevalues class androidx.mediarouter.** {
static boolean DEBUG return false;
}
# Copyright 2019 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.
# Help R8 to remove debug related coded. All of these static constants are
# initialized to: Log.isLoggable(DEBUG, TAG).
-assumevalues class android.support.v7.app.MediaRouteControllerDialogFragment {
static boolean USE_SUPPORT_DYNAMIC_GROUP return false;
}
-assumevalues class android.support.v7.media.MediaRouter {
static boolean DEBUG return false;
}
-assumevalues class android.support.v7.media.RegisteredMediaRouteProvider {
static boolean DEBUG return false;
}
...@@ -250,6 +250,14 @@ class BuildConfigGenerator extends DefaultTask { ...@@ -250,6 +250,14 @@ class BuildConfigGenerator extends DefaultTask {
// Target has AIDL, but we don't support it yet: http://crbug.com/644439 // Target has AIDL, but we don't support it yet: http://crbug.com/644439
sb.append(' ignore_aidl = true\n') sb.append(' ignore_aidl = true\n')
break break
case 'com_android_support_mediarouter_v7':
sb.append(' # https://crbug.com/1000382\n')
sb.append(' proguard_configs = ["support_mediarouter.flags"]\n')
break
case 'androidx_mediarouter_mediarouter':
sb.append(' # https://crbug.com/1000382\n')
sb.append(' proguard_configs = ["androidx_mediarouter.flags"]\n')
break
case 'androidx_transition_transition': case 'androidx_transition_transition':
// Not specified in the POM, compileOnly dependency not supposed to be used unless // Not specified in the POM, compileOnly dependency not supposed to be used unless
// the library is present: b/70887421 // the library is present: b/70887421
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment