Closed Bug 1332752 Opened 8 years ago Closed 8 years ago

Add some logging to TimeoutManager

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

Details

Attachments

(1 file)

I'm going to add some logging which can be helpful with the upcoming manual tests on TimeoutManager.
Attachment #8829021 - Flags: review?(bkelly)
Comment on attachment 8829021 [details] [diff] [review] Add some logging to TimeoutManager Review of attachment 8829021 [details] [diff] [review]: ----------------------------------------------------------------- I'm not a huge fan of permanent log statements like this, but ok.
Attachment #8829021 - Flags: review?(bkelly) → review+
(In reply to Ben Kelly [:bkelly] from comment #2) > I'm not a huge fan of permanent log statements like this, but ok. May I ask why out of curiosity?
(In reply to :Ehsan Akhgari from comment #4) > May I ask why out of curiosity? Having spent a lot of time on these in the past I found that: 1) Each person tends to want different messages 2) Putting them all in makes things spammy 3) Consequently they are a net negative compared to people just writing a printf when they need it But like I said, its just a preference and I'm not blocking you.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: