Refactored SmartsFlags and friends...again
SmartsFlags was initially extending RawFlags and overriding some
non-final methods by adding some validation logic and calling super,
which increased the process size (as hundred of methods woudl have
2 implementations, on SmartFlags and RawFlags).
This CL "fixes" the issue by moving the non-final methods out of
RawFlags and:
- Changing SmartFlags to call mBackend instead of super
- Moving the non-final methods to a new RawFlagsForTests class.
Test: atest AdServicesServiceCoreUnitTests --test-filter='.*Flags.*Test'
Bug: 384798806
Bug: 391406689
Fixes: 386415138
Flag: TEST_ONLY
BYPASS_PS_FLAG_CHECK: test-only change
Test: AdServicesServiceCoreUnitTests --test-filter='.*Flags.*Test'
Change-Id: Id477ad49a1d2d21705cd5fd5edf12aea15108151
6 files changed