commit ba33e133c76f27f13c691890d108260ad090afdb
author Anton Hansson <hansson@google.com> Tue Aug 18 12:52:51 2020 +0100
committer Anton Hansson <hansson@google.com> Fri Sep 11 13:21:32 2020 +0100
tree 8583985d8238f755645337827b9b4a5996a1f820
parent 82c362963a3744a1b880e8a2321de5cc61ffd5b5

Simplify module visibility post build refactor

//visibility:override is no longer needed for impl_library_visibility
to override visibility.

Removing this allows the defaults module to specify better defaults.

- Stub libraries are made publicly visible, via `visibility`
- Impl libraries are private by default, but visibility is extended
  by the modules

Bug: 165017290
Test: m
Exempt-From-Owner-Approval: build refactor
Change-Id: Ibf35bfac5c99a21125f89ba10945f3364217b90f
Merged-In: Ibf35bfac5c99a21125f89ba10945f3364217b90f
(cherry picked from commit 05ce161d4e7d26a0057ccd7e1755115ade92664b)

framework/Android.bp

diff --git a/framework/Android.bp b/framework/Android.bp
index 941ff61..3040041 100644
--- a/framework/Android.bp
+++ b/framework/Android.bp
@@ -129,12 +129,8 @@
     },
     hostdex: true, // for hiddenapi check
 
-    // Allow access to the stubs from anywhere.
-    visibility: ["//visibility:public"],
-
     // Restrict access to implementation library.
     impl_library_visibility: [
-        "//visibility:override", // Ignore the visibility property.
         "//frameworks/opt/net/wifi/service:__subpackages__",
     ] + test_access_hidden_api_whitelist,