Skip to content

Commit

Permalink
[atoms] minimize size of get-attribute atom (SeleniumHQ#13416)
Browse files Browse the repository at this point in the history
Co-authored-by: Diego Molina <[email protected]>
  • Loading branch information
joerg1985 and diemol authored Jan 10, 2024
1 parent d51c74a commit ff82c4a
Showing 1 changed file with 16 additions and 2 deletions.
18 changes: 16 additions & 2 deletions javascript/webdriver/atoms/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,20 @@ closure_js_library(
],
)

closure_js_library(
name = "attribute-only",
srcs = ["attribute.js"],
suppress = [
"JSC_IMPLICITLY_NULLABLE_JSDOC",
"JSC_NULLABLE_RETURN_WITH_NAME",
],
visibility = ["//visibility:private"],
deps = [
"//javascript/atoms:domcore",
"@io_bazel_rules_closure//closure/library",
],
)

closure_fragment(
name = "clear-local-storage",
function = "webdriver.atoms.storage.local.clear",
Expand Down Expand Up @@ -83,7 +97,7 @@ closure_fragment(
closure_fragment(
name = "get-attribute",
function = "webdriver.atoms.element.attribute.get",
module = "webdriver.atoms.element",
module = "webdriver.atoms.element.attribute",
visibility = [
"//dotnet/src/webdriver:__pkg__",
"//java/src/org/openqa/selenium/remote:__pkg__",
Expand All @@ -94,7 +108,7 @@ closure_fragment(
"//rb/lib/selenium/webdriver/atoms:__pkg__",
],
deps = [
":atoms-lib",
":attribute-only",
],
)

Expand Down

0 comments on commit ff82c4a

Please sign in to comment.