browser: Fixed that no logs output on IE8/9

[mooyoul]

IE 8/9 reports console methods as objects when using the typeof operator.
https://web.archive.org/web/20111123190115/http://whattheheadsaid.com/2011/04/internet-explorer-9s-problematic-console-object

Closes #148.

[TooTallNate]

Maybe we just check for console.log truthiness instead? (i.e. ditch the typeof for console.log)

[mooyoul]

I think that's not necessary because if console.log really plain Object,
An error will be thrown during execute below line:

Function.prototype.apply.call(console.log, console, arguments);

Like this:

[TooTallNate]

Sure, but typeof conslole.log == 'object' will still be true with your patch as well.

[mooyoul]

Hmm, What you think about replacing typeof for console.log to truthiness check?

[mooyoul]

... and Removing return keyword from https://github.com/visionmedia/debug/blob/master/browser.js#L106 ?

I think it is not necessary because that statement (below code / https://github.com/visionmedia/debug/blob/master/browser.js#L108) always returns undefined.

Function.prototype.apply.call(console.log, console, arguments);

So, I think that return keyword to be deleted...

[TooTallNate]

Hmm, What you think about replacing typeof for console.log to truthiness check?

Yes, that's what I mentioned up above 😄

[TooTallNate]

Let's not worry about the return part for now. Though you're technically correct, let's stay focused on the IE fix here.

[mooyoul]

Agree 👍 I'll commit that changes :)

[mooyoul]

Okay, I just committed that change, and tested on major web browsers. (IE8, IE9, IE10, IE11, Chrome Latest, Chrome Canary Latest, Firefox Latest, Safari Latest, Android 2.3.3, iOS 7, iOS 8, IE Mobile 11, etc...)
It works like a charm. and code looks more DRY 😄

[TooTallNate]

Thanks!

[mooyoul]

You're welcome. Thank you!