Closed
Bug 1219470
Opened 9 years ago
Closed 9 years ago
Replace PRLogModuleInfo usage with LazyLogModule in layout
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla45
People
(Reporter: erahm, Assigned: sajitk)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
This covers replacing PRLogModuleInfo w/ LazyLogModule in the 'layout/' directory.
Current usage:
> ./layout/base/AccessibleCaretLogger.cpp:17: log = PR_NewLogModule("AccessibleCaret");
> ./layout/base/LayoutLogging.cpp:15: log = PR_NewLogModule("layout");
> ./layout/base/nsDocumentViewer.cpp:144: sLog = PR_NewLogModule("printing");
> ./layout/base/nsPresShell.cpp:767: gLog = PR_NewLogModule("PresShell");
> ./layout/base/nsRefreshDriver.cpp:739: gLog = PR_NewLogModule("nsRefreshDriver");
> ./layout/base/SelectionCarets.cpp:89: gSelectionCaretsLog = PR_NewLogModule(kSelectionCaretsLogModuleName);
> ./layout/base/TouchCaret.cpp:83: gTouchCaretLog = PR_NewLogModule(kTouchCaretLogModuleName);
> ./layout/generic/nsFlexContainerFrame.cpp:41: sLog = PR_NewLogModule("nsFlexContainerFrame");
> ./layout/generic/nsFrame.cpp:227: gStyleVerifyTreeLogModuleInfo = PR_NewLogModule("styleverifytree");
> ./layout/generic/nsFrame.cpp:244: gLogModule = PR_NewLogModule("frame");
> ./layout/generic/nsPluginFrame.cpp:102: sLog = PR_NewLogModule("nsPluginFrame");
> ./layout/generic/nsSimplePageSequenceFrame.cpp:50: sLog = PR_NewLogModule("printing-layout");
> ./layout/printing/nsPrintData.cpp:25: sLog = PR_NewLogModule("printing");
> ./layout/printing/nsPrintEngine.cpp:148: sLog = PR_NewLogModule("printing");
> ./layout/style/Loader.cpp:274: sLog = PR_NewLogModule("nsCSSLoader");
> ./layout/style/Loader.cpp:283: gSriPRLog = PR_NewLogModule("SRI");
Attachment #8683924 -
Flags: review?(roc)
Attachment #8683924 -
Flags: review?(roc) → review+
Keywords: checkin-needed
Keywords: checkin-needed
|
||
Comment 4•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•