| 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;
|
|
|
|
|