Element::isFocusable() debug asserts too frequently
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 5 Apr 2019 01:09:44 +0000 (01:09 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 5 Apr 2019 01:09:44 +0000 (01:09 +0000)
https://bugs.webkit.org/show_bug.cgi?id=196634

Reviewed by Geoffrey Garen.

Remove the debug assertion for now.

* dom/Element.cpp:
(WebCore::Element::isFocusable const):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243923 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/dom/Element.cpp

index 014e98b..a2c10e3 100644 (file)
@@ -1,3 +1,15 @@
+2019-04-04  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Element::isFocusable() debug asserts too frequently
+        https://bugs.webkit.org/show_bug.cgi?id=196634
+
+        Reviewed by Geoffrey Garen.
+
+        Remove the debug assertion for now.
+
+        * dom/Element.cpp:
+        (WebCore::Element::isFocusable const):
+
 2019-04-04  Simon Fraser  <simon.fraser@apple.com>
 
         Have ScrollableArea store a ScrollType for the current scroll
index a423625..bb59e3d 100644 (file)
@@ -553,7 +553,8 @@ bool Element::isFocusable() const
     if (!renderer()) {
         // If the node is in a display:none tree it might say it needs style recalc but
         // the whole document is actually up to date.
-        ASSERT(!needsStyleRecalc() || !document().childNeedsStyleRecalc());
+        // FIXME: We should be able to assert !needsStyleRecalc() || !document().childNeedsStyleRecalc()
+        // but it hits too frequently on websites like Gmail and Microsoft Exchange.
 
         // Elements in canvas fallback content are not rendered, but they are allowed to be
         // focusable as long as their canvas is displayed and visible.