This reverts commit 988487cf.

Reason for revert: Breaks git_master-art: b/285863158

Change-Id: I9421f905535839a857991f3b90bc3e85a9b74074
Bug: 285863158

The framework-res manifest defines a number of class entrypoints
implemented in modules. These required manual keep rules to prevent
shrinking, but now we can use the aapt-generated keep rules to achieve
the same result, without the maintenance burden.

Bug: 272495195
Test: m + verify no net change in module dex
Change-Id: Icdbbd7f8626a31ab619fff26e7fe979b61c74e61

service-art is not under path packages/modules/. This change will make
optimization setting visible to service-art

Bug: 247779319
Test: presubmit
Change-Id: Iff2b24f2c2f92c5e0c80c177bf911ce94d3733cd

Instead of including standalone system server keep rules in
system-api.pro, this change add system-server.pro file to the file group
of system server optimization.

Bug: 247779319
Test: presubmit and Run MTS locally with command `mts-tradefed run` with
`mts-permission`.

Change-Id: Ief2cf3a9e8995a379738d4a70fdb2280a8447bb9

Extending default optimization settings by enabling R8 full mode, disabling ignore warnings, and keeping all classes
extending SystemService.

Bug: 247779319
Test: presubmit and Run MTS locally with command `mts-tradefed run` with
`mts-permission`.

Change-Id: I27a615c6bba39815001a31561fe8313eb7cbd367

This reverts commit 340aca97.

Reason for revert: The original dependency issue was fixed

Bug: 242597723
Test: Build mainline modules locally. Build should fail if warnings
occur.

Change-Id: I8451fb8be3914bbc652ff11a62bf9ff39152b5f1

This reverts commit 5e7fab69.

Reason for revert: Dependency changes are not checked-in. Revert this change for now.

Change-Id: If267081b677e34a983c29aec2eceba2db7471099

Disable ignore warnings by default for all standalone system server jar
optimization to avoid potential issues.

Bug: 242597723
Test: Build mainline modules locally. Build should fail if warnings
occur.

Change-Id: I30af90fb9421f2fb313fe35360229516da482a75

Before the change, the keep rule only keeps the class that extends
SystemServer, but if the methods/fields in the class are not used by
other entries, then the methods/fields will be removed. It will cause
issue for some service are not only used by calling the public methods
inherited from SystemServer. This change will keep all methods, and
fields in the class that extends SystemServer.

Bug: 233746531
Test: Build modules using this configuration locally, and check all
neseccary methods are kept.

Change-Id: Ia938fe8726ac8d4c1f2066f1ebca318b624e30de

packages/modules

Before common proguard configuration is only visible to modules in its
subpackage, and Permission module. This configuration file will be
shared by all mainline modules. So this change makes it visible to all
modules under packages/modules path.

Test: Refer to this configuration in module target, and use m to build
module locally.
Bug: 233746531

Change-Id: Ib97403fa272b74e6b3ef9e8f641b233447aebc91

modules

Add common optimization settings for standalone system server modules.

Test: m
Bug: 233746531
Change-Id: I181999e03b5b338238f4f250ce38be395e68816b

Bug: 182523293
Test: build
Change-Id: Ie8b2bff1521a05d53071a9bd34c240d4624cdca8