add logging support for exceptions

add logging support for exceptions

BUG=
R=gram@google.com

Committed: https://code.google.com/p/dart/source/detail?r=24246

[Siggi Cherem (dart-lang), 2013-06-20 00:40:29]

ptal

[gram, 2013-06-20 00:49:23]

lgtm

https://codereview.chromium.org/17467008/diff/3001/pkg/logging/lib/logging.dart
File pkg/logging/lib/logging.dart (right):

https://codereview.chromium.org/17467008/diff/3001/pkg/logging/lib/logging.da...
pkg/logging/lib/logging.dart:320: LogRecord(this.level, this.message,
this.loggerName, [time, this.exception])
this.loggerName, DateTime time, ...

https://codereview.chromium.org/17467008/diff/3001/pkg/logging/test/logging_t...
File pkg/logging/test/logging_test.dart (right):

https://codereview.chromium.org/17467008/diff/3001/pkg/logging/test/logging_t...
pkg/logging/test/logging_test.dart:272: });
It feels like this should be driven by the message list, splitting each message
on space and using that to drive the logging. You could use the same driver for
the previous test. Maybe just a TODO for now.

[Siggi Cherem (dart-lang), 2013-06-20 00:57:10]

https://codereview.chromium.org/17467008/diff/3001/pkg/logging/lib/logging.dart
File pkg/logging/lib/logging.dart (right):

https://codereview.chromium.org/17467008/diff/3001/pkg/logging/lib/logging.da...
pkg/logging/lib/logging.dart:320: LogRecord(this.level, this.message,
this.loggerName, [time, this.exception])
On 2013/06/20 00:49:24, gram wrote:
> this.loggerName, DateTime time, ...
> 

Done - remove the time argument, it is not really used.

[Siggi Cherem (dart-lang), 2013-06-20 19:43:12]

Committed patchset #3 manually as r24246 (presubmit successful).