Closed Bug 1755966 Opened 10 months ago Closed 9 months ago

telemetry client tests that crash firefox on Windows don't have crash dump output


(Toolkit :: Telemetry, defect, P3)




100 Branch
Tracking Status
firefox100 --- fixed


(Reporter: willkg, Assigned: b4hand)



(1 file)

When I was working on bug #1594515, I noticed that I could get crash processing output from the stackwalker on Linux and macOS, but not on Windows.

The output_timeout value we're talking about is set here:

It's set to 300 (seconds), but raising it to 1000 (seconds) gives Windows machinery enough time to recognize that it crashed and run the minidump through minidump-stackwalk and get crash processing output.

Between bug #1594515 and bug #1746747, we've fixed symbolication of crash stacks, but we still have to increase the output_timeout so we get stacks on Windows.

Assignee: nobody → bforehand

Is there a way to test this change on try or locally? I am assuming no.

Flags: needinfo?(willkg)

I tested it out by forcing a crash in the telemetry test client code:

diff --git a/toolkit/components/telemetry/tests/marionette/harness/telemetry_harness/ b/toolkit/components/telemetry/tests/
--- a/toolkit/components/telemetry/tests/marionette/harness/telemetry_harness/
+++ b/toolkit/components/telemetry/tests/marionette/harness/telemetry_harness/
@@ -72,6 +72,13 @@ class TelemetryTestCase(WindowManagerMix
+    def crash_browser(self):
+        """Crash the browser."""
+        # FIXME(willkg): remove this
+        with self.marionette.using_context(self.marionette.CONTEXT_CONTENT):
+            self.marionette.navigate("about:crashparent")
     def search(self, text):
         """Perform a search via the browser's URL bar."""
diff --git a/toolkit/components/telemetry/tests/marionette/tests/client/ b/toolkit/components/telemetry/tests/
--- a/toolkit/components/telemetry/tests/marionette/tests/client/
+++ b/toolkit/components/telemetry/tests/marionette/tests/client/
@@ -12,6 +12,9 @@ class TestMainTabScalars(TelemetryTestCa
     def test_main_tab_scalars(self):
         """Test for Telemetry Scalars."""
+        # Crash the browser
+        self.crash_browser()
         with self.marionette.using_context(self.marionette.CONTEXT_CHROME):
             start_tab = self.marionette.current_window_handle
Flags: needinfo?(willkg)

The severity field is not set for this bug.
:chutten, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(chutten)
Severity: -- → S4
Flags: needinfo?(chutten)
Priority: -- → P3

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:b4hand, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(willkg)
Flags: needinfo?(bforehand)

I don't have level 3 commit access, so I asked chutten to kindly press the button to land this.

Flags: needinfo?(willkg)
Flags: needinfo?(bforehand)
Pushed by
Increase timeout to gather logs during a crash.; r=willkg,chutten
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.