Benjamin Stover (:stechz) Reporter
	Description • 13 years ago

It's printfing all over the place!

	Benjamin Stover (:stechz) Reporter
	Comment 2 • 13 years ago

Compile issues on Windows

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 3 • 13 years ago

Comment on attachment 533684 [details] [diff] [review]
Make TabChild be quiet

Is anyone going to use log which just prints "creating ..." and "loading..."?

Would it make sense to put those printfs just inside #ifdef DEBUG?

	Benjamin Stover (:stechz) Reporter
	Comment 4 • 13 years ago

I just want them hidden by default, debug and optimized. Perhaps remove them if they aren't useful to anyone?

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 5 • 13 years ago

Comment on attachment 533684 [details] [diff] [review]
Make TabChild be quiet

So unless there is some good use case for the log, I'd prefer
#ifdef DEBUG

	Benjamin Stover (:stechz) Reporter
	Comment 6 • 13 years ago

(In reply to comment #5)
> Comment on attachment 533684 [details] [diff] [review] [review]
> Make TabChild be quiet
> 
> So unless there is some good use case for the log, I'd prefer
> #ifdef DEBUG

Again, I don't want these logs on debug builds either.

	Benjamin Stover (:stechz) Reporter
	Comment 7 • 13 years ago

At least, not enabled by default.

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
	Comment 8 • 13 years ago

I can't imagine these printfs being useful to anyone anymore.  Some of them were added for debugging purposes during the initial e10s sprint in 2009 ...

	Tom S [:evilpie] Assignee
	Comment 9 • 11 years ago

I think now that we have e10s for thumbs we should really fix this logging in release builds.

	Tom S [:evilpie] Assignee
	Comment 10 • 11 years ago

I don't think these log message are helpful in any way, so we might as well remove them.

	Ryan VanderMeulen [:RyanVM]
	Comment 11 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/4d2adaf69288