commit 3bb009ac148d6e1978d9233b0131a027585752f7
author Martin Stjernholm <mast@google.com> Thu Oct 17 21:29:01 2019 +0100
committer Treehugger Robot <treehugger-gerrit@google.com> Mon Oct 21 14:34:02 2019 +0000
tree f5177cffa33c1e6479a04d438cce937306ad8dfe
parent e41eec3b193d594577cbe8afc7c3dc6608c2f201

Use art_defaults in libnative*, for consistency.

Also change to //art:__submodules__ for consistent internal visibility, and
some TODO comment corrections.

Test: m WITH_TIDY=1
Test: build & boot
Test: atest art/libnativeloader/test (no new errors, but existing ones on arm64 - b/142896688)
Test: (atest art/libnativebridge/tests doesn't work at all, but that's not new - b/142897320)
Bug: 137364733
Bug: 133140750
Change-Id: If9ddb5a597eea3606342c3625dcdb3bcf5d19041

libnativebridge/Android.bp

diff --git a/libnativebridge/Android.bp b/libnativebridge/Android.bp
index 2dd92d1..f2db117 100644
--- a/libnativebridge/Android.bp
+++ b/libnativebridge/Android.bp
@@ -1,9 +1,6 @@
 cc_defaults {
     name: "libnativebridge-defaults",
-    cflags: [
-        "-Werror",
-        "-Wall",
-    ],
+    defaults: ["art_defaults"],
     cppflags: [
         "-fvisibility=protected",
     ],
@@ -22,9 +19,8 @@
     name: "libnativebridge",
     defaults: ["libnativebridge-defaults"],
     visibility: [
-        "//art:__subpackages__",
-        // TODO(b/133140750): Clean this up.
         "//frameworks/base/cmds/app_process",
+        // TODO(b/133140750): Clean this up.
         "//frameworks/base/native/webview/loader/libwebviewchromium_loader",
     ],