Remove unused resource overrides

AFAICT these resource overrides are not used, but (if accessed
by library clients) can lead to crashes.

R.dimen.abc_action_bar_progress_bar_size doesn't appear to be
used anywhere.

R.bool.abc_action_bar_embed_tabs is only accessed in appcompat,
where it also has definitions both in values/ and values-port/.
I'm removing the lone declaration in compat/values-port/. Perhaps
this is an accidental leftover from the code moving?

These were found by a new lint check in 3.2; a generalized version
of the ExtraTranslations check which flags resources that appear
only in resource overrides, but not in the base folder (with some
special cases.)

This is the issue explanation:

   Explanation for issues of type "MissingDefaultResource":
   If a resource is only defined in folders with qualifiers like -land or -en,
   and there is no default declaration in the base folder (values or layout
   etc), then the app will crash if that resource is accessed on a device
   where the device is in a configuration missing the given qualifier.

   As a special case, drawables do not have to be specified in the base
   folder; if there is a match in a density folder (such as drawable-mdpi)
   that image will be used and scaled. Note however that if you  only specify
   a drawable in a folder like drawable-en-hdpi, the app will crash in
   non-English locales.

   There may be scenarios where you have a resource, such as a -fr drawable,
   which is only referenced from some other resource with the same qualifiers
   (such as a -fr style), which itself has safe fallbacks. However, this still
   makes it possible for somebody to accidentally reference the drawable and
   crash, so it is safer to create a default dummy fallback in the base
   folder. Alternatively, you can suppress the issue by adding
   tools:ignore="MissingDefaultResource" on the element.

   (This scenario frequently happens with string translations, where you might
   delete code and the corresponding resources, but forget to delete a
   translation. There is a dedicated issue id for that scenario, with the id
   ExtraTranslation.)

Change-Id: I52b01819998671fec019b148d9536ec0b80c39a4
2 files changed
tree: 56b15c397d854ac6b7cdec47d3a92d9b33a60154
  1. .idea/
  2. annotations/
  3. api/
  4. app-toolkit/
  5. asynclayoutinflater/
  6. buildSrc/
  7. car/
  8. collections/
  9. compat/
  10. content/
  11. coordinatorlayout/
  12. core-ui/
  13. core-utils/
  14. cursoradapter/
  15. customtabs/
  16. customview/
  17. design/
  18. development/
  19. documentfile/
  20. drawerlayout/
  21. dynamic-animation/
  22. emoji/
  23. exifinterface/
  24. fragment/
  25. frameworks/
  26. gradle/
  27. graphics/
  28. heifwriter/
  29. interpolator/
  30. jetifier/
  31. leanback/
  32. lifecycle/
  33. loader/
  34. localbroadcastmanager/
  35. media-compat/
  36. paging/
  37. percent/
  38. persistence/
  39. preference-leanback/
  40. print/
  41. recommendation/
  42. recyclerview-selection/
  43. room/
  44. samples/
  45. scripts/
  46. slices/
  47. slidingpanelayout/
  48. swiperefreshlayout/
  49. testutils/
  50. textclassifier/
  51. transition/
  52. tv-provider/
  53. v13/
  54. v14/
  55. v4/
  56. v7/
  57. viewpager/
  58. viewpager2/
  59. wear/
  60. webkit/
  61. webkit-codegen/
  62. .gitignore
  63. adding-support-library-as-included-build.md
  64. Android.mk
  65. build.gradle
  66. CleanSpec.mk
  67. gradle.properties
  68. gradlew
  69. include-composite-deps.gradle
  70. include-support-library.gradle
  71. LICENSE.txt
  72. makeFlatfootRepo.sh
  73. OWNERS
  74. pathmap.mk
  75. PREUPLOAD.cfg
  76. README.md
  77. settings.gradle
README.md

AOSP Support Library Contribution Guide

Accepted Types of Contributions

  • Bug fixes (needs a corresponding bug report in b.android.com)
  • Each bug fix is expected to come with tests
  • Fixing spelling errors
  • Updating documentation
  • Adding new tests to the area that is not currently covered by tests

We are not currently accepting new modules, features, or behavior changes.

Checking Out the Code

NOTE: You will need to use Linux or Mac OS. Building under Windows is not currently supported.

Follow the “Downloading the Source” guide to install and set up repo tool, but instead of running the listed repo commands to initialize the repository, run the folowing:

repo init -u https://android.googlesource.com/platform/manifest -b ub-supportlib-master

Now your repository is set to pull only what you need for building and running support library. Download the code (and grab a coffee while we pull down 7GB):

repo sync -j8 -c

You will use this command to sync your checkout in the future - it’s similar to git fetch

Using Android Studio

Open path/to/checkout/frameworks/support/ in Android Studio. Now you're ready edit, run, and test!

If you get “Unregistered VCS root detected” click “Add root” to enable git integration for Android Studio.

If you see any warnings (red underlines) run Build > Clean Project.

Builds

Full Build (Optional)

You can do most of your work from Android Studio, however you can also build the full support library from command line:

cd path/to/checkout/frameworks/support/
./gradlew createArchive

Building Support Library as part of your App build

If you intend to repeatedly make changes to Support Library and to wish to see the results in your app, and you don't want to have to repeatedly build them as separate Gradle projects, you can configure your app build to build Support Library too

Running Tests

Single Test Class or Method

  1. Open the desired test file in Android Studio.
  2. Right-click on a test class or @Test method name and select Run FooBarTest

Full Test Package

  1. In the project side panel open the desired module.
  2. Find the directory with the tests
  3. Right-click on the directory and select Run android.support.foobar

Running Sample Apps

Support library has a set of Android applications that exercise support library code. These applications can be useful when you want to debug a real running application, or reproduce a problem interactively, before writing test code.

These applications are named support-*-demos (e.g. support-4v-demos or support-leanback-demos. You can run them by clicking Run > Run ... and choosing the desired application.

Making a change

cd path/to/checkout/frameworks/support/
repo start my_branch_name .
(make needed modifications)
git commit -a
repo upload --current-branch .

If you see the following prompt, choose always:

Run hook scripts from https://android.googlesource.com/platform/manifest (yes/always/NO)?

Getting reviewed

  • After you run repo upload, open r.android.com
  • Sign in into your account (or create one if you do not have one yet)
  • Add an appropriate reviewer (use git log to find who did most modifications on the file you are fixing)