diff options
| author | 2019-10-11 11:30:01 +0100 | |
|---|---|---|
| committer | 2019-10-14 07:49:45 +0000 | |
| commit | c78860b91ae07eed92f86876e7a03132adea6fcd (patch) | |
| tree | 820f338333010f4d6e4b543ffea47e9ff7d2dd3f | |
| parent | b9a8541380c2bdb9685e16d8d4b2ba034ddbf870 (diff) | |
Remove temporary soong visibility workarounds
Removes temporary unrestricted visibility for libnativeloader and
libnativebridge.
Bug: 137364733
Test: m nothing
Change-Id: I32d871b639deddd664c42e9962f6826ab798862f
| -rw-r--r-- | libnativebridge/Android.bp | 16 | ||||
| -rw-r--r-- | libnativeloader/Android.bp | 23 |
2 files changed, 29 insertions, 10 deletions
diff --git a/libnativebridge/Android.bp b/libnativebridge/Android.bp index c97845d16a..2dd92d117e 100644 --- a/libnativebridge/Android.bp +++ b/libnativebridge/Android.bp @@ -21,8 +21,12 @@ cc_library_headers { cc_library { name: "libnativebridge", defaults: ["libnativebridge-defaults"], - // TODO(oth): remove after moving under art/ (b/137364733) - visibility: ["//visibility:public"], + visibility: [ + "//art:__subpackages__", + // TODO(b/133140750): Clean this up. + "//frameworks/base/cmds/app_process", + "//frameworks/base/native/webview/loader/libwebviewchromium_loader", + ], host_supported: true, srcs: ["native_bridge.cc"], @@ -54,8 +58,12 @@ cc_library { cc_library { name: "libnativebridge_lazy", defaults: ["libnativebridge-defaults"], - // TODO(oth): remove after moving under art/ (b/137364733) - visibility: ["//visibility:public"], + visibility: [ + "//art/libnativebridge/tests", + "//frameworks/base/core/jni", + "//frameworks/native/opengl/libs", + "//frameworks/native/vulkan/libvulkan", + ], host_supported: false, srcs: ["native_bridge_lazy.cc"], diff --git a/libnativeloader/Android.bp b/libnativeloader/Android.bp index 4b9c27f26b..f73becc41e 100644 --- a/libnativeloader/Android.bp +++ b/libnativeloader/Android.bp @@ -16,8 +16,12 @@ cc_defaults { cc_library { name: "libnativeloader", defaults: ["libnativeloader-defaults"], - // TODO(oth): remove after moving under art/ (b/137364733) - visibility: ["//visibility:public"], + visibility: [ + "//art:__subpackages__", + // TODO(b/133140750): Clean this up. + "//frameworks/base/cmds/app_process", + "//frameworks/base/native/webview/loader", + ], host_supported: true, srcs: [ "native_loader.cpp", @@ -54,8 +58,11 @@ cc_library { cc_library { name: "libnativeloader_lazy", defaults: ["libnativeloader-defaults"], - // TODO(oth): remove after moving under art/ (b/137364733) - visibility: ["//visibility:public"], + visibility: [ + "//frameworks/base/core/jni", + "//frameworks/native/opengl/libs", + "//frameworks/native/vulkan/libvulkan", + ], host_supported: false, srcs: ["native_loader_lazy.cpp"], required: ["libnativeloader"], @@ -63,8 +70,12 @@ cc_library { cc_library_headers { name: "libnativeloader-headers", - // TODO(oth): remove after moving under art/ (b/137364733) - visibility: ["//visibility:public"], + visibility: [ + "//art/libnativeloader/test", + "//frameworks/av/media/libstagefright", + "//frameworks/native/libs/graphicsenv", + "//frameworks/native/vulkan/libvulkan", + ], host_supported: true, export_include_dirs: ["include"], } |