779625 - Send telemetry metric when exceptions are thrown.

Status: RESOLVED WONTFIX

(Tamarin Graveyard :: Profiler, defect)

Description

[George Comninos]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_0) AppleWebKit/537.1 (KHTML, like Gecko) Chrome/21.0.1180.57 Safari/537.1

Steps to reproduce:

We should send a metric for thrown exceptions. It should include the exception type (i.e. class) and the stack trace. Many users have asked for the ability to get the stack trace in release builds when exceptions are thrown.

Comment 1

[George Comninos]

Attachment: Send metric for thrown exceptions with stack trace

Attached patch for: 1) add method for dumping the MethodFrame stack to a string, and 2) send a telemetry metric for thrown exceptions with the error class and stack trace.

Comment 2

[Ruchi Lohani]

Comment on attachment 648069 [details] [diff] [review]
Send metric for thrown exceptions with stack trace

Review of attachment 648069 [details] [diff] [review]:
-----------------------------------------------------------------

::: //depot/main/FlashRuntime/Main/code/third_party/avmplus/core/AvmCore.cpp
@@ +1656,5 @@
> +                // get the full class name including the namespace
> +                so->traits()->print(exceptionStringBuffer, true);
> +            } else {
> +                // backup, just convert atom to a string, it will say "Error"
> +                exceptionStringBuffer << StUTF8String(string(exception->atom)).c_str();

I did not fully understand the comment regarding "Error".

Comment 3

[George Comninos]

The comment just means it will say just "Error" for the exception class instead of something more useful like "flash.errors::IOError". Interestingly "Error" is all the exception dialog in debugger builds says. I don't actually know of any cases where it would get into the backup code anyway.

Comment 4

[Tamarin Bot]

changeset: 7528:cbe55af57d88
user:      Dan Schaffer <[e-mail]>
summary:      Bug 779625: Send telemetry metric when exceptions are thrown. (p=gcomnino,r=fklockii,r=rulohani)

http://hg.mozilla.org/tamarin-redux/rev/cbe55af57d88

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Tamarin isn't maintained anymore. WONTFIX remaining bugs.