Log Message
updateRendering trace point should exclude SVG https://bugs.webkit.org/show_bug.cgi?id=214649
Reviewed by Geoffrey Garen. Don't issue the trace point for SVGImage Pages. * page/Page.cpp: (WebCore::Page::updateRendering):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (264777 => 264778)
--- trunk/Source/WebCore/ChangeLog 2020-07-23 18:19:26 UTC (rev 264777)
+++ trunk/Source/WebCore/ChangeLog 2020-07-23 18:25:20 UTC (rev 264778)
@@ -1,3 +1,15 @@
+2020-07-23 Simon Fraser <simon.fra...@apple.com>
+
+ updateRendering trace point should exclude SVG
+ https://bugs.webkit.org/show_bug.cgi?id=214649
+
+ Reviewed by Geoffrey Garen.
+
+ Don't issue the trace point for SVGImage Pages.
+
+ * page/Page.cpp:
+ (WebCore::Page::updateRendering):
+
2020-07-23 Zalan Bujtas <za...@apple.com>
Unreviewed, reverting r262124.
Modified: trunk/Source/WebCore/page/Page.cpp (264777 => 264778)
--- trunk/Source/WebCore/page/Page.cpp 2020-07-23 18:19:26 UTC (rev 264777)
+++ trunk/Source/WebCore/page/Page.cpp 2020-07-23 18:25:20 UTC (rev 264778)
@@ -1424,7 +1424,9 @@
return;
}
- TraceScope traceScope(RenderingUpdateStart, RenderingUpdateEnd);
+ bool isSVGImagePage = chrome().client().isSVGImageChromeClient();
+ if (!isSVGImagePage)
+ tracePoint(RenderingUpdateStart);
SetForScope<bool> change(m_inUpdateRendering, true);
@@ -1494,6 +1496,9 @@
layoutIfNeeded();
doAfterUpdateRendering();
+
+ if (!isSVGImagePage)
+ tracePoint(RenderingUpdateEnd);
}
void Page::doAfterUpdateRendering()
_______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes
