Index: chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc |
diff --git a/chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc b/chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc |
index c8c7fada128e363657e48157f2c542500b3272df..a8c8bffc85b18c6e34acd9615cd9320f2d712969 100644 |
--- a/chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc |
+++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc |
@@ -562,26 +562,13 @@ |
if (!rfh) |
return VISIBILITY_ERROR; |
- // clang-format off |
- std::string jsFindVisibility = R"( |
- (function isNodeVisible(node) { |
- if (!node) return 'node not found'; |
- if (node.offsetWidth === 0 || node.offsetHeight === 0) return false; |
- // Don't check opacity, since the css transition may actually leave |
- // opacity at 0 after it's been unhidden |
- if (node.classList.contains('hidden')) return false; |
- // Otherwise, we must check all parent nodes |
- var parentVisibility = isNodeVisible(node.parentElement); |
- if (parentVisibility === 'node not found') { |
- return true; // none of the parents are set invisible |
- } |
- return parentVisibility; |
- }(document.getElementById(')" + node_id + R"(')));)"; |
- // clang-format on |
- |
- std::unique_ptr<base::Value> value = |
- content::ExecuteScriptAndGetValue(rfh, jsFindVisibility); |
- |
+ std::unique_ptr<base::Value> value = content::ExecuteScriptAndGetValue( |
+ rfh, "var node = document.getElementById('" + node_id + |
+ "');\n" |
+ "if (node)\n" |
+ " node.offsetWidth > 0 && node.offsetHeight > 0;" |
+ "else\n" |
+ " 'node not found';\n"); |
if (!value.get()) |
return VISIBILITY_ERROR; |