nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1899894 - Remove debug statements that require customized #define. r=layout-reviewers,emilio

Browse source 
The debug statements all require customized #define to enable. I don't feel it
is useful to leave code that is not compiled in the default debug build.

Differential Revision: https://phabricator.services.mozilla.com/D212192
This commit is contained in:
Ting-Yu Lin 2024-05-31 17:45:44 +00:00
parent b170aa854c
commit 7e69da1e37
6 changed files with 0 additions and 69 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

7
layout/base/PresShell.cpp
View file

@ -1693,10 +1693,6 @@ void PresShell::EndObservingDocument() {
mIsObservingDocument = false;
}
#ifdef DEBUG_kipp
char* nsPresShell_ReflowStackPointerTop;
#endif
void PresShell::InitPaintSuppressionTimer() {
// Default to PAINTLOCK_EVENT_DELAY if we can't get the pref value.
Document* doc = mDocument->GetDisplayDocument()
@ -10894,9 +10890,6 @@ void ReflowCountMgr::DoGrandHTMLTotals() {
//------------------------------------
void ReflowCountMgr::DisplayTotals(const char* aStr) {
# ifdef DEBUG_rods
printf("%s\n", aStr ? aStr : "No name");
# endif
if (mDumpFrameCounts) {
DoGrandTotals();
}

4
layout/forms/nsListControlFrame.cpp
View file

@ -806,10 +806,6 @@ nsListControlFrame::DoneAddingChildren(bool aIsDone) {
NS_IMETHODIMP
nsListControlFrame::AddOption(int32_t aIndex) {
#ifdef DO_REFLOW_DEBUG
printf("---- Id: %d nsLCF %p Added Option %d\n", mReflowId, this, aIndex);
#endif
if (!mIsAllContentHere) {
mIsAllContentHere = Select().IsDoneAddingChildren();
if (!mIsAllContentHere) {

15
layout/forms/nsListControlFrame.h
View file

@ -6,17 +6,6 @@
#ifndef nsListControlFrame_h___
#define nsListControlFrame_h___
#ifdef DEBUG_evaughan
// #define DEBUG_rods
#endif
#ifdef DEBUG_rods
// #define DO_REFLOW_DEBUG
// #define DO_REFLOW_COUNTER
// #define DO_UNCONSTRAINED_CHECK
// #define DO_PIXELS
#endif
#include "mozilla/Attributes.h"
#include "mozilla/ScrollContainerFrame.h"
#include "mozilla/StaticPtr.h"
@ -339,10 +328,6 @@ class nsListControlFrame final : public mozilla::ScrollContainerFrame,
RefPtr<mozilla::HTMLSelectEventListener> mEventListener;
static nsListControlFrame* mFocused;
#ifdef DO_REFLOW_COUNTER
int32_t mReflowId;
#endif
};
#endif /* nsListControlFrame_h___ */

22
layout/generic/nsBlockFrame.cpp
View file

@ -1923,11 +1923,6 @@ nsReflowStatus nsBlockFrame::TrialReflow(nsPresContext* aPresContext,
if (HasOverflowLines() || HasPushedFloats()) {
state.mReflowStatus.SetNextInFlowNeedsReflow();
}
#ifdef DEBUG_kipp
ListTag(stdout);
printf(": block is not fully complete\n");
#endif
}
// Place the ::marker's frame if it is placed next to a block child.
@ -2338,14 +2333,6 @@ nscoord nsBlockFrame::ComputeFinalSize(const ReflowInput& aReflowInput,
aMetrics.SetSize(wm, finalSize);
#ifdef DEBUG_blocks
if ((ABSURD_SIZE(aMetrics.Width()) || ABSURD_SIZE(aMetrics.Height())) &&
!GetParent()->IsAbsurdSizeAssertSuppressed()) {
ListTag(stdout);
printf(": WARNING: desired:%d,%d\n", aMetrics.Width(), aMetrics.Height());
}
#endif
return blockEndEdgeOfChildren;
}
@ -8132,15 +8119,6 @@ void nsBlockFrame::CheckFloats(BlockReflowState& aState) {
NS_ERROR(
"nsBlockFrame::CheckFloats: Explicit float list is out of sync with "
"float cache");
# if defined(DEBUG_roc)
nsIFrame::RootFrameList(PresContext(), stdout, 0);
for (i = 0; i < lineFloats.Length(); ++i) {
printf("Line float: %p\n", lineFloats.ElementAt(i));
}
for (i = 0; i < storedFloats.Length(); ++i) {
printf("Stored float: %p\n", storedFloats.ElementAt(i));
}
# endif
}
#endif

17
layout/mathml/nsMathMLmtableFrame.cpp
View file

@ -622,23 +622,6 @@ static void ParseAlignAttribute(nsString& aValue, eAlign& aAlign,
}
}
#ifdef DEBUG_rbs_off
// call ListMathMLTree(mParent) to get the big picture
static void ListMathMLTree(nsIFrame* atLeast) {
// climb up to <math> or <body> if <math> isn't there
nsIFrame* f = atLeast;
for (; f; f = f->GetParent()) {
nsIContent* c = f->GetContent();
if (!c || c->IsMathMLElement(nsGkAtoms::math) ||
// XXXbaku which kind of body tag?
c->NodeInfo()->NameAtom(nsGkAtoms::body))
break;
}
if (!f) f = atLeast;
f->List(stdout, 0);
}
#endif
// --------
// implementation of nsMathMLmtableWrapperFrame

4
layout/xul/nsXULTooltipListener.cpp
View file

@ -73,10 +73,6 @@ void nsXULTooltipListener::MouseOut(Event* aEvent) {
return;
}
#ifdef DEBUG_crap
if (mNeedTitletip) return;
#endif
// check to see if the mouse left the targetNode, and if so,
// hide the tooltip
if (currentTooltip) {