Skip to content

Commit 6b24560

Browse files
committed
[bazel] Upgrade closure rules to 0.11.0
1 parent 83b2f46 commit 6b24560

File tree

12 files changed

+91
-18
lines changed

12 files changed

+91
-18
lines changed

WORKSPACE

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -78,19 +78,6 @@ load("@maven//:defs.bzl", "pinned_maven_install")
7878

7979
pinned_maven_install()
8080

81-
http_archive(
82-
name = "io_bazel_rules_closure",
83-
sha256 = "2e95ba060acd74f3662547a38814ffff60317be047b7168d25498aea52f3e732",
84-
strip_prefix = "rules_closure-b3d4ec3879620edcadd3422b161cebb37c59b6c5",
85-
urls = [
86-
"https://github.com/bazelbuild/rules_closure/archive/b3d4ec3879620edcadd3422b161cebb37c59b6c5.tar.gz",
87-
],
88-
)
89-
90-
load("@io_bazel_rules_closure//closure:defs.bzl", "closure_repositories")
91-
92-
closure_repositories()
93-
9481
http_archive(
9582
name = "d2l_rules_csharp",
9683
sha256 = "4e8e9a93a5436d81c0a410148bef9e3e44860cd7dc567dd4732ab5b269a5e1f8",
@@ -139,6 +126,21 @@ pip_install(
139126
requirements = "//py:requirements.txt",
140127
)
141128

129+
http_archive(
130+
name = "io_bazel_rules_closure",
131+
sha256 = "d66deed38a0bb20581c15664f0ab62270af5940786855c7adc3087b27168b529",
132+
strip_prefix = "rules_closure-0.11.0",
133+
urls = [
134+
"https://github.com/bazelbuild/rules_closure/archive/0.11.0.tar.gz",
135+
],
136+
)
137+
138+
load("@io_bazel_rules_closure//closure:repositories.bzl", "rules_closure_dependencies", "rules_closure_toolchains")
139+
140+
rules_closure_dependencies()
141+
142+
rules_closure_toolchains()
143+
142144
http_archive(
143145
name = "rules_pkg",
144146
sha256 = "aeca78988341a2ee1ba097641056d168320ecc51372ef7ff8e64b139516a4937",

javascript/atoms/BUILD.bazel

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ closure_js_library(
2626
"JSC_IMPLICITLY_NULLABLE_JSDOC",
2727
"JSC_STRICT_INEXISTENT_PROPERTY",
2828
"JSC_UNKNOWN_EXPR_TYPE",
29+
"JSC_USE_OF_GOOG_PROVIDE",
2930
],
3031
visibility = [
3132
"//javascript/atoms/fragments:__pkg__",
@@ -46,11 +47,17 @@ closure_js_library(
4647
closure_js_library(
4748
name = "bot",
4849
srcs = ["bot.js"],
50+
suppress = [
51+
"JSC_USE_OF_GOOG_PROVIDE",
52+
],
4953
)
5054

5155
closure_js_library(
5256
name = "color",
5357
srcs = ["color.js"],
58+
suppress = [
59+
"JSC_USE_OF_GOOG_PROVIDE",
60+
],
5461
deps = [
5562
"@io_bazel_rules_closure//closure/library",
5663
],
@@ -70,6 +77,7 @@ closure_js_library(
7077
"JSC_NULLABLE_RETURN_WITH_NAME",
7178
"JSC_STRICT_INEXISTENT_PROPERTY",
7279
"JSC_UNKNOWN_EXPR_TYPE",
80+
"JSC_USE_OF_GOOG_PROVIDE",
7381
],
7482
deps = [
7583
":bot",
@@ -89,6 +97,7 @@ closure_js_library(
8997
"JSC_IMPLICITLY_NULLABLE_JSDOC",
9098
"JSC_STRICT_INEXISTENT_PROPERTY",
9199
"JSC_UNKNOWN_EXPR_TYPE",
100+
"JSC_USE_OF_GOOG_PROVIDE",
92101
],
93102
deps = [
94103
":errors",
@@ -105,6 +114,7 @@ closure_js_library(
105114
"JSC_IMPLICITLY_NULLABLE_JSDOC",
106115
"JSC_STRICT_INEXISTENT_PROPERTY",
107116
"JSC_UNKNOWN_EXPR_TYPE",
117+
"JSC_USE_OF_GOOG_PROVIDE",
108118
],
109119
deps = [
110120
":bot",
@@ -128,6 +138,7 @@ closure_js_library(
128138
"JSC_MISSING_CONST_PROPERTY",
129139
"JSC_STRICT_INEXISTENT_PROPERTY",
130140
"JSC_UNKNOWN_EXPR_TYPE",
141+
"JSC_USE_OF_GOOG_PROVIDE",
131142
],
132143
)
133144

@@ -139,6 +150,7 @@ closure_js_library(
139150
"JSC_MISSING_CONST_PROPERTY",
140151
"JSC_STRICT_INEXISTENT_PROPERTY",
141152
"JSC_UNKNOWN_EXPR_TYPE",
153+
"JSC_USE_OF_GOOG_PROVIDE",
142154
],
143155
deps = [
144156
":bot",
@@ -157,6 +169,7 @@ closure_js_library(
157169
"JSC_IMPLICITLY_NULLABLE_JSDOC",
158170
"JSC_NULLABLE_RETURN_WITH_NAME",
159171
"JSC_UNKNOWN_EXPR_TYPE",
172+
"JSC_USE_OF_GOOG_PROVIDE",
160173
],
161174
visibility = [
162175
"//javascript/webdriver/atoms/inject:__pkg__",
@@ -177,6 +190,7 @@ closure_js_library(
177190
"JSC_IMPLICITLY_NULLABLE_JSDOC",
178191
"JSC_NULLABLE_RETURN_WITH_NAME",
179192
"JSC_UNKNOWN_EXPR_TYPE",
193+
"JSC_USE_OF_GOOG_PROVIDE",
180194
],
181195
deps = [
182196
":bot",
@@ -194,6 +208,7 @@ closure_js_library(
194208
"JSC_IMPLICITLY_NULLABLE_JSDOC",
195209
"JSC_STRICT_INEXISTENT_PROPERTY",
196210
"JSC_UNKNOWN_EXPR_TYPE",
211+
"JSC_USE_OF_GOOG_PROVIDE",
197212
],
198213
visibility = [
199214
"//javascript/atoms/fragments:__pkg__",
@@ -211,6 +226,9 @@ closure_js_library(
211226
closure_js_library(
212227
name = "json",
213228
srcs = ["json.js"],
229+
suppress = [
230+
"JSC_USE_OF_GOOG_PROVIDE",
231+
],
214232
deps = [
215233
":useragent",
216234
"@io_bazel_rules_closure//closure/library",
@@ -228,6 +246,7 @@ closure_js_library(
228246
"JSC_LATE_PROVIDE_ERROR",
229247
"JSC_NULLABLE_RETURN_WITH_NAME",
230248
"JSC_UNKNOWN_EXPR_TYPE",
249+
"JSC_USE_OF_GOOG_PROVIDE",
231250
],
232251
deps = [
233252
":bot",
@@ -246,6 +265,7 @@ closure_js_library(
246265
suppress = [
247266
"JSC_STRICT_INEXISTENT_PROPERTY",
248267
"JSC_UNKNOWN_EXPR_TYPE",
268+
"JSC_USE_OF_GOOG_PROVIDE",
249269
],
250270
deps = ["@io_bazel_rules_closure//closure/library"],
251271
)
@@ -256,6 +276,9 @@ closure_js_library(
256276
"frame.js",
257277
"window.js",
258278
],
279+
suppress = [
280+
"JSC_USE_OF_GOOG_PROVIDE",
281+
],
259282
deps = [
260283
":bot",
261284
":errors",
@@ -272,6 +295,7 @@ closure_js_library(
272295
"JSC_IMPLICITLY_NULLABLE_JSDOC",
273296
"JSC_STRICT_INEXISTENT_PROPERTY",
274297
"JSC_UNKNOWN_EXPR_TYPE",
298+
"JSC_USE_OF_GOOG_PROVIDE",
275299
],
276300
deps = [
277301
":bot",
@@ -285,6 +309,9 @@ closure_js_library(
285309
name = "test_util",
286310
testonly = 1,
287311
srcs = ["test/test_util.js"],
312+
suppress = [
313+
"JSC_USE_OF_GOOG_PROVIDE",
314+
],
288315
deps = [
289316
":useragent",
290317
"@io_bazel_rules_closure//closure/library",

javascript/atoms/dom.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -574,7 +574,7 @@ bot.dom.isShown = function(elem, opt_ignoreOpacity) {
574574
/**
575575
* Determines whether an element or its parents have `display: none` set
576576
* @param {!Node} e the element
577-
* @return {boolean}
577+
* @return {!boolean}
578578
*/
579579
function displayed(e) {
580580
if (bot.dom.isElement(e)) {
@@ -609,7 +609,7 @@ bot.dom.isShown = function(elem, opt_ignoreOpacity) {
609609
return false;
610610
}
611611

612-
return parent && displayed(parent);
612+
return !!parent && displayed(parent);
613613
}
614614

615615
return bot.dom.isShown_(elem, !!opt_ignoreOpacity, displayed);

javascript/chrome-driver/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ closure_js_library(
1010
"JSC_PREFER_BACKTICKS_TO_AT_SIGN_CODE",
1111
"JSC_STRICT_INEXISTENT_PROPERTY",
1212
"JSC_UNKNOWN_EXPR_TYPE",
13+
"JSC_USE_OF_GOOG_PROVIDE",
1314
],
1415
deps = [
1516
"//javascript/atoms:dom",

javascript/ie-driver/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ closure_js_library(
1616
],
1717
suppress = [
1818
"JSC_UNKNOWN_EXPR_TYPE",
19+
"JSC_USE_OF_GOOG_PROVIDE",
1920
],
2021
deps = [
2122
"//javascript/atoms:dom",

javascript/remote/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ closure_js_library(
2424
"JSC_PREFER_BACKTICKS_TO_AT_SIGN_CODE",
2525
"JSC_STRICT_INEXISTENT_PROPERTY",
2626
"JSC_UNKNOWN_EXPR_TYPE",
27+
"JSC_USE_OF_GOOG_PROVIDE",
2728
],
2829
deps = [
2930
"//javascript/atoms:errors",

javascript/selenium-atoms/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,14 @@ closure_js_library(
1717
"JSC_IMPLICITLY_NULLABLE_JSDOC",
1818
"JSC_MISSING_JSDOC",
1919
"JSC_MISSING_PARAMETER_JSDOC",
20-
"JSC_MISSING_REQUIRE_STRICT_WARNING",
2120
"JSC_MISSING_RETURN_JSDOC",
2221
"JSC_MUST_BE_PRIVATE",
2322
"JSC_PREFER_BACKTICKS_TO_AT_SIGN_CODE",
2423
"JSC_PRIMITIVE_OBJECT",
2524
"JSC_STRICT_INEXISTENT_PROPERTY",
2625
"JSC_UNKNOWN_EXPR_TYPE",
2726
"JSC_UNUSED_LOCAL_ASSIGNMENT",
27+
"JSC_USE_OF_GOOG_PROVIDE",
2828
],
2929
deps = [
3030
"//javascript/atoms:action",

javascript/webdriver/BUILD.bazel

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ closure_js_library(
1515
"JSC_IMPLICITLY_NULLABLE_JSDOC",
1616
"JSC_MISSING_CONST_PROPERTY",
1717
"JSC_UNKNOWN_EXPR_TYPE",
18+
"JSC_USE_OF_GOOG_PROVIDE",
1819
],
1920
visibility = ["//javascript/remote:__pkg__"],
2021
deps = [
@@ -26,13 +27,19 @@ closure_js_library(
2627
closure_js_library(
2728
name = "key",
2829
srcs = ["key.js"],
30+
suppress = [
31+
"JSC_USE_OF_GOOG_PROVIDE",
32+
],
2933
visibility = ["//javascript:__subpackages__"],
3034
)
3135

3236
closure_js_library(
3337
name = "all_js_for_testing",
3438
testonly = 1,
3539
srcs = glob(["**/*.js"]),
40+
suppress = [
41+
"JSC_USE_OF_GOOG_PROVIDE",
42+
],
3643
visibility = ["//javascript:__pkg__"],
3744
deps = ["@io_bazel_rules_closure//closure/library"],
3845
)

javascript/webdriver/atoms/BUILD.bazel

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ closure_js_library(
2727
suppress = [
2828
"JSC_IMPLICITLY_NULLABLE_JSDOC",
2929
"JSC_NULLABLE_RETURN_WITH_NAME",
30+
"JSC_USE_OF_GOOG_PROVIDE",
3031
],
3132
visibility = ["//javascript:__subpackages__"],
3233
deps = [
@@ -44,6 +45,9 @@ closure_fragment(
4445
name = "clear-local-storage",
4546
function = "webdriver.atoms.storage.local.clear",
4647
module = "webdriver.atoms.storage.local",
48+
suppress = [
49+
"JSC_USE_OF_GOOG_PROVIDE",
50+
],
4751
visibility = [
4852
"//javascript/chrome-driver:__pkg__",
4953
],
@@ -267,6 +271,9 @@ closure_js_library(
267271
name = "all_js_for_testing",
268272
testonly = 1,
269273
srcs = glob(["**/*.js"]),
274+
suppress = [
275+
"JSC_USE_OF_GOOG_PROVIDE",
276+
],
270277
visibility = [
271278
"//javascript:__pkg__",
272279
"//javascript/chrome-driver:__pkg__",

javascript/webdriver/atoms/inject/BUILD.bazel

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ closure_js_library(
3131
suppress = [
3232
"JSC_IMPLICITLY_NULLABLE_JSDOC",
3333
"JSC_UNKNOWN_EXPR_TYPE",
34+
"JSC_USE_OF_GOOG_PROVIDE",
3435
],
3536
deps = [
3637
":execute-script",
@@ -44,6 +45,9 @@ closure_js_library(
4445
closure_js_library(
4546
name = "appcache",
4647
srcs = ["appcache.js"],
48+
suppress = [
49+
"JSC_USE_OF_GOOG_PROVIDE",
50+
],
4751
deps = [
4852
"//javascript/atoms:inject",
4953
"//javascript/webdriver/atoms:atoms-lib",
@@ -56,6 +60,7 @@ closure_js_library(
5660
suppress = [
5761
"JSC_IMPLICITLY_NULLABLE_JSDOC",
5862
"JSC_UNKNOWN_EXPR_TYPE",
63+
"JSC_USE_OF_GOOG_PROVIDE",
5964
],
6065
deps = [
6166
":execute-script",
@@ -73,6 +78,7 @@ closure_js_library(
7378
suppress = [
7479
"JSC_IMPLICITLY_NULLABLE_JSDOC",
7580
"JSC_NULLABLE_RETURN_WITH_NAME",
81+
"JSC_USE_OF_GOOG_PROVIDE",
7682
],
7783
deps = [
7884
"//javascript/atoms:inject",
@@ -85,6 +91,7 @@ closure_js_library(
8591
suppress = [
8692
"JSC_IMPLICITLY_NULLABLE_JSDOC",
8793
"JSC_UNKNOWN_EXPR_TYPE",
94+
"JSC_USE_OF_GOOG_PROVIDE",
8895
],
8996
deps = [
9097
":execute-script",
@@ -100,6 +107,7 @@ closure_js_library(
100107
suppress = [
101108
"JSC_IMPLICITLY_NULLABLE_JSDOC",
102109
"JSC_NULLABLE_RETURN_WITH_NAME",
110+
"JSC_USE_OF_GOOG_PROVIDE",
103111
],
104112
deps = [
105113
":execute-script",
@@ -110,6 +118,9 @@ closure_js_library(
110118
closure_js_library(
111119
name = "local-storage",
112120
srcs = ["local_storage.js"],
121+
suppress = [
122+
"JSC_USE_OF_GOOG_PROVIDE",
123+
],
113124
deps = [
114125
":execute-script",
115126
"//javascript/webdriver/atoms:atoms-lib",
@@ -119,6 +130,9 @@ closure_js_library(
119130
closure_js_library(
120131
name = "session-storage",
121132
srcs = ["session_storage.js"],
133+
suppress = [
134+
"JSC_USE_OF_GOOG_PROVIDE",
135+
],
122136
deps = [
123137
":execute-script",
124138
"//javascript/webdriver/atoms:atoms-lib",
@@ -128,6 +142,9 @@ closure_js_library(
128142
closure_js_library(
129143
name = "sql-database",
130144
srcs = ["sql_database.js"],
145+
suppress = [
146+
"JSC_USE_OF_GOOG_PROVIDE",
147+
],
131148
deps = [
132149
":execute-script",
133150
"//javascript/atoms:errors",

0 commit comments

Comments
 (0)