refactor: broaden visibility and use list() instead of keys() (#1704)

This upstreams a couple trivial Google patches to make rules_python more
compatible with some Google-internal changes.

* Use list(dict) instead of dict.keys(). This is because some bzl files
are run through a Python-based testing framework, and dict.keys() can't
be concatenated to a list in Python 3.
* Expand visibility to all of rules_python instead of just the python
subdirectory. This makes some patches that use internals easier to
maintain, but also makes the visiblity more in line with the rest of the
project (where `//:__subpackages__` is used for convenience, as it
avoids having to change visibility frequently, but still prevents public
dependencies).

python/private/common/BUILD.bazel

@@ -15,7 +15,7 @@
 load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
 
 package(
-    default_visibility = ["//python:__subpackages__"],
+    default_visibility = ["//:__subpackages__"],
 )
 
 bzl_library(

python/private/common/attributes.bzl

@@ -226,7 +226,7 @@ COMMON_ATTR_NAMES = [
     "testonly",
     "toolchains",
     "visibility",
-] + COMMON_ATTRS.keys()
+] + list(COMMON_ATTRS)  # Use list() instead .keys() so it's valid Python
 
 # Attribute names common to all test=True rules
 TEST_ATTR_NAMES = COMMON_ATTR_NAMES + [
@@ -236,10 +236,10 @@ TEST_ATTR_NAMES = COMMON_ATTR_NAMES + [
     "flaky",
     "shard_count",
     "local",
-] + AGNOSTIC_TEST_ATTRS.keys()
+] + list(AGNOSTIC_TEST_ATTRS)  # Use list() instead .keys() so it's valid Python
 
 # Attribute names common to all executable=True rules
 BINARY_ATTR_NAMES = COMMON_ATTR_NAMES + [
     "args",
     "output_licenses",  # NOTE: Common to all rules, but slated for removal
-] + AGNOSTIC_BINARY_ATTRS.keys()
+] + list(AGNOSTIC_BINARY_ATTRS)  # Use list() instead .keys() so it's valid Python