Closed Bug 763138 Opened 8 years ago Closed 8 years ago

Telemetry should indicate whether a debugger is attached

Categories

(Toolkit :: Telemetry, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: vladan, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

This would help prevent red herrings when investigating performance problems and chrome hangs. On Windows, this would be as simple as calling the IsDebuggerPresent method.
Attachment #635059 - Flags: feedback?(taras.mozilla)
Comment on attachment 635059 [details] [diff] [review]
Report to Telemetry whether a debugger is attached

>+      gWasDebuggerPresent = debugService.isDebuggerPresent;

s/DebuggerPresent/DebuggerAttached/ everywhere in the patch sounds nicer to me.

>+
>+  if (sysctl(mib, mibSize, &info, &infoSize, NULL, 0)) {
>+    return NS_ERROR_FAILURE;

Don't do this, return NS_OK & false. This will cause telemetry to fail since there is no try/catch there. Afaik OSX stuff like this is implemented as IPC internally and will occasionally fail.
Attachment #635059 - Flags: feedback?(taras.mozilla) → feedback+
Applied Taras's comments
Attachment #635059 - Attachment is obsolete: true
Attachment #635110 - Flags: review?(benjamin)
Comment on attachment 635110 [details] [diff] [review]
Report to Telemetry whether a debugger is attached

requesting review from smichaud for the mac impl of nsDebugImpl::GetIsDebuggerAttached.
Attachment #635110 - Flags: review?(smichaud)
Attachment #635110 - Flags: review?(benjamin)
Attachment #635110 - Flags: review+
Comment on attachment 635110 [details] [diff] [review]
Report to Telemetry whether a debugger is attached

Looks fine to me.
Attachment #635110 - Flags: review?(smichaud) → review+
Comment on attachment 635110 [details] [diff] [review]
Report to Telemetry whether a debugger is attached

https://hg.mozilla.org/integration/mozilla-inbound/rev/23d9d354236e
Attachment #635110 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/23d9d354236e
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Depends on: 1016631
You need to log in before you can comment on or make changes to this bug.