diff options
| author | 2024-01-16 20:02:00 +0000 | |
|---|---|---|
| committer | 2024-01-16 20:20:27 +0000 | |
| commit | 78b6b30aec81d3028fdcd6f5ca39d2fb30eca06b (patch) | |
| tree | 7991225ff92ea9472a4378a40fb6f775390274ed /java | |
| parent | d06bdb031042ea448cc2a34fae9ccc4a50f63434 (diff) | |
Remove HideFlaggedApi() related logic and the related product variables
With the recent build changes to support generating mainlin module sdk
with flagged apis, the build no longer depends on the values of these
product variables in exposing the flagged apis, but these are determined
by the aconfig flags. Given that these variables are no longer used,
this change removes these variables and the variables dependent code.
Test: m nothing --no-skip-soong-tests
Bug: 320515715
Change-Id: I6af94da73cc7fc7ffce670928aad81cec5d383b4
Diffstat (limited to 'java')
| -rw-r--r-- | java/config/droidstubs.go | 9 | ||||
| -rw-r--r-- | java/droidstubs.go | 3 | ||||
| -rw-r--r-- | java/droidstubs_test.go | 34 |
3 files changed, 0 insertions, 46 deletions
diff --git a/java/config/droidstubs.go b/java/config/droidstubs.go index 7029fee06..f46c893fd 100644 --- a/java/config/droidstubs.go +++ b/java/config/droidstubs.go @@ -55,13 +55,6 @@ var ( } MetalavaAnnotationsWarningsFlags = strings.Join(metalavaAnnotationsWarningsFlags, " ") - - metalavaHideFlaggedApis = []string{ - "--revert-annotation", - "android.annotation.FlaggedApi", - } - - MetalavaHideFlaggedApis = strings.Join(metalavaHideFlaggedApis, " ") ) const ( @@ -73,8 +66,6 @@ func init() { exportedVars.ExportString("MetalavaAddOpens", MetalavaAddOpens) - exportedVars.ExportStringList("MetalavaHideFlaggedApis", metalavaHideFlaggedApis) - exportedVars.ExportStringListStaticVariable("MetalavaAnnotationsFlags", metalavaAnnotationsFlags) exportedVars.ExportStringListStaticVariable("MetalavaAnnotationWarningsFlags", metalavaAnnotationsWarningsFlags) diff --git a/java/droidstubs.go b/java/droidstubs.go index bdbaf9281..9d3188b01 100644 --- a/java/droidstubs.go +++ b/java/droidstubs.go @@ -710,9 +710,6 @@ func metalavaCmd(ctx android.ModuleContext, rule *android.RuleBuilder, javaVersi } cmd.Flag(config.MetalavaFlags) - if ctx.DeviceConfig().HideFlaggedApis() { - cmd.Flag(config.MetalavaHideFlaggedApis) - } return cmd } diff --git a/java/droidstubs_test.go b/java/droidstubs_test.go index 379e2403f..52cd1c513 100644 --- a/java/droidstubs_test.go +++ b/java/droidstubs_test.go @@ -22,8 +22,6 @@ import ( "testing" "android/soong/android" - - "github.com/google/blueprint/proptools" ) func TestDroidstubs(t *testing.T) { @@ -364,38 +362,6 @@ func TestGeneratedApiContributionVisibilityTest(t *testing.T) { ctx.ModuleForTests("bar", "android_common") } -func TestDroidstubsHideFlaggedApi(t *testing.T) { - result := android.GroupFixturePreparers( - prepareForJavaTest, - android.FixtureModifyProductVariables(func(variables android.FixtureProductVariables) { - variables.NextReleaseHideFlaggedApi = proptools.BoolPtr(true) - variables.Release_expose_flagged_api = proptools.BoolPtr(false) - }), - android.FixtureMergeMockFs(map[string][]byte{ - "a/A.java": nil, - "a/current.txt": nil, - "a/removed.txt": nil, - }), - ).RunTestWithBp(t, ` - droidstubs { - name: "foo", - srcs: ["a/A.java"], - api_surface: "public", - check_api: { - current: { - api_file: "a/current.txt", - removed_api_file: "a/removed.txt", - } - }, - } - `) - - m := result.ModuleForTests("foo", "android_common") - manifest := m.Output("metalava.sbox.textproto") - cmdline := String(android.RuleBuilderSboxProtoForTests(t, result.TestContext, manifest).Commands[0].Command) - android.AssertStringDoesContain(t, "flagged api hide command not included", cmdline, "--revert-annotation android.annotation.FlaggedApi") -} - func TestAconfigDeclarations(t *testing.T) { result := android.GroupFixturePreparers( prepareForJavaTest, |