Bug 1675409 - Migrated TracingMarkerPayload to Markers 2.0 API - r=gregtatum

Differential Revision: https://phabricator.services.mozilla.com/D96053
This commit is contained in:
Gerald Squelart
2020-11-17 22:25:06 +00:00
parent e5ba951c98
commit d1fe64af1f
16 changed files with 162 additions and 237 deletions

View File

@@ -129,6 +129,10 @@
#include "nsTextNode.h"
#include "ActiveLayerTracker.h"
#ifdef MOZ_GECKO_PROFILER
# include "mozilla/ProfilerMarkerTypes.h"
#endif
using namespace mozilla;
using namespace mozilla::dom;
@@ -8194,9 +8198,8 @@ static nsIFrame* FindPreviousNonWhitespaceSibling(nsIFrame* aFrame) {
bool nsCSSFrameConstructor::MaybeRecreateContainerForFrameRemoval(
nsIFrame* aFrame) {
#define TRACE(reason) \
PROFILER_TRACING_MARKER("Layout", \
"MaybeRecreateContainerForFrameRemoval: " reason, \
LAYOUT, TRACING_EVENT)
PROFILER_MARKER("MaybeRecreateContainerForFrameRemoval: " reason, LAYOUT, \
{}, Tracing, "Layout")
MOZ_ASSERT(aFrame, "Must have a frame");
MOZ_ASSERT(aFrame->GetParent(), "Frame shouldn't be root");
MOZ_ASSERT(aFrame == aFrame->FirstContinuation(),
@@ -10719,9 +10722,8 @@ bool nsCSSFrameConstructor::MaybeRecreateForColumnSpan(
// some of them have column-span:all descendants. Sadly, there's no way to
// detect this by checking FrameConstructionItems in WipeContainingBlock().
// Otherwise, we would have already wiped the multi-column containing block.
PROFILER_TRACING_MARKER(
"Layout", "Reframe multi-column after constructing frame list", LAYOUT,
TRACING_EVENT);
PROFILER_MARKER("Reframe multi-column after constructing frame list",
LAYOUT, {}, Tracing, "Layout");
// aFrameList can contain placeholder frames. In order to destroy their
// associated out-of-flow frames properly, we need to manually flush all the
@@ -11046,9 +11048,9 @@ static bool IsSafeToAppendToIBSplitInline(nsIFrame* aParentFrame,
}
bool nsCSSFrameConstructor::WipeInsertionParent(nsContainerFrame* aFrame) {
#define TRACE(reason) \
PROFILER_TRACING_MARKER("Layout", "WipeInsertionParent: " reason, LAYOUT, \
TRACING_EVENT)
#define TRACE(reason) \
PROFILER_MARKER("WipeInsertionParent: " reason, LAYOUT, {}, Tracing, \
"Layout");
const LayoutFrameType frameType = aFrame->Type();
@@ -11108,9 +11110,9 @@ bool nsCSSFrameConstructor::WipeContainingBlock(
nsFrameConstructorState& aState, nsIFrame* aContainingBlock,
nsIFrame* aFrame, FrameConstructionItemList& aItems, bool aIsAppend,
nsIFrame* aPrevSibling) {
#define TRACE(reason) \
PROFILER_TRACING_MARKER("Layout", "WipeContainingBlock: " reason, LAYOUT, \
TRACING_EVENT)
#define TRACE(reason) \
PROFILER_MARKER("WipeContainingBlock: " reason, LAYOUT, {}, Tracing, \
"Layout");
if (aItems.IsEmpty()) {
return false;