Document::needsStyleRecalc() shouldn't return true for fragment scrolling (m_gotoAnchorNeededAfterStylesheetsLoad)
https://bugs.webkit.org/show_bug.cgi?id=204593
Reviewed by Antti Koivisto.
After r252761, we no longer triggers a fragment scrolling as a part of style resolution.
Consequently, there is no need for Document::needsStyleRecalc() to return true
even when m_gotoAnchorNeededAfterStylesheetsLoad is set and there are no more pending stylesheets.
No new tests since there shouldn't really be any observable behavior difference.
* dom/Document.cpp:
(WebCore::Document::needsStyleRecalc const):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@252881 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index b819a23..2f7614d 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,19 @@
+2019-11-26 Ryosuke Niwa <rniwa@webkit.org>
+
+ Document::needsStyleRecalc() shouldn't return true for fragment scrolling (m_gotoAnchorNeededAfterStylesheetsLoad)
+ https://bugs.webkit.org/show_bug.cgi?id=204593
+
+ Reviewed by Antti Koivisto.
+
+ After r252761, we no longer triggers a fragment scrolling as a part of style resolution.
+ Consequently, there is no need for Document::needsStyleRecalc() to return true
+ even when m_gotoAnchorNeededAfterStylesheetsLoad is set and there are no more pending stylesheets.
+
+ No new tests since there shouldn't really be any observable behavior difference.
+
+ * dom/Document.cpp:
+ (WebCore::Document::needsStyleRecalc const):
+
2019-11-26 Antoine Quint <graouts@apple.com>
[Web Animations] Layout of children of element with forwards-filling opacity animation may be incorrect after removal
diff --git a/Source/WebCore/dom/Document.cpp b/Source/WebCore/dom/Document.cpp
index b4cc56b..d562e93 100644
--- a/Source/WebCore/dom/Document.cpp
+++ b/Source/WebCore/dom/Document.cpp
@@ -2032,10 +2032,6 @@
if (styleScope().hasPendingUpdate())
return true;
- // Ensure this happens eventually as it is currently in resolveStyle. This can be removed if the code moves.
- if (m_gotoAnchorNeededAfterStylesheetsLoad && !styleScope().hasPendingSheets())
- return true;
-
return false;
}
