Title: [280700] trunk/Source/WebCore
- Revision
- 280700
- Author
- [email protected]
- Date
- 2021-08-05 11:03:13 -0700 (Thu, 05 Aug 2021)
Log Message
Document::isLayoutTimerActive should read isLayoutPending
https://bugs.webkit.org/show_bug.cgi?id=228835
Reviewed by Antti Koivisto.
Callers of this function curious about whether there's a layout scheduled.
* dom/Document.cpp:
(WebCore::Document::isLayoutPending const):
(WebCore::Document::isLayoutTimerActive const): Deleted.
* dom/Document.h:
* html/parser/HTMLParserScheduler.cpp:
(WebCore::HTMLParserScheduler::continueNextChunkTimerFired):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (280699 => 280700)
--- trunk/Source/WebCore/ChangeLog 2021-08-05 17:18:19 UTC (rev 280699)
+++ trunk/Source/WebCore/ChangeLog 2021-08-05 18:03:13 UTC (rev 280700)
@@ -1,3 +1,19 @@
+2021-08-05 Alan Bujtas <[email protected]>
+
+ Document::isLayoutTimerActive should read isLayoutPending
+ https://bugs.webkit.org/show_bug.cgi?id=228835
+
+ Reviewed by Antti Koivisto.
+
+ Callers of this function curious about whether there's a layout scheduled.
+
+ * dom/Document.cpp:
+ (WebCore::Document::isLayoutPending const):
+ (WebCore::Document::isLayoutTimerActive const): Deleted.
+ * dom/Document.h:
+ * html/parser/HTMLParserScheduler.cpp:
+ (WebCore::HTMLParserScheduler::continueNextChunkTimerFired):
+
2021-08-05 Michael Catanzaro <[email protected]>
Suppress more -Wreturn-type warnings
Modified: trunk/Source/WebCore/dom/Document.cpp (280699 => 280700)
--- trunk/Source/WebCore/dom/Document.cpp 2021-08-05 17:18:19 UTC (rev 280699)
+++ trunk/Source/WebCore/dom/Document.cpp 2021-08-05 18:03:13 UTC (rev 280700)
@@ -3246,7 +3246,7 @@
return true;
}
-bool Document::isLayoutTimerActive() const
+bool Document::isLayoutPending() const
{
return view() && view()->layoutContext().isLayoutPending();
}
Modified: trunk/Source/WebCore/dom/Document.h (280699 => 280700)
--- trunk/Source/WebCore/dom/Document.h 2021-08-05 17:18:19 UTC (rev 280699)
+++ trunk/Source/WebCore/dom/Document.h 2021-08-05 18:03:13 UTC (rev 280700)
@@ -735,7 +735,7 @@
bool parsing() const { return m_bParsing; }
bool shouldScheduleLayout() const;
- bool isLayoutTimerActive() const;
+ bool isLayoutPending() const;
#if !LOG_DISABLED
Seconds timeSinceDocumentCreation() const { return MonotonicTime::now() - m_documentCreationTime; };
#endif
Modified: trunk/Source/WebCore/html/parser/HTMLParserScheduler.cpp (280699 => 280700)
--- trunk/Source/WebCore/html/parser/HTMLParserScheduler.cpp 2021-08-05 17:18:19 UTC (rev 280699)
+++ trunk/Source/WebCore/html/parser/HTMLParserScheduler.cpp 2021-08-05 18:03:13 UTC (rev 280700)
@@ -91,7 +91,7 @@
// FIXME: The timer class should handle timer priorities instead of this code.
// If a layout is scheduled, wait again to let the layout timer run first.
- if (m_parser.document()->isLayoutTimerActive()) {
+ if (m_parser.document()->isLayoutPending()) {
m_continueNextChunkTimer.startOneShot(0_s);
return;
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
