New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 595766 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Starting tracing from Telemetry is not robust to preceding failures

Project Member Reported by petrcermak@chromium.org, Mar 17 2016

Issue description

If a Telemetry test that uses tracing fails in the middle of the tracing session (without crashing the browser), all subsequent Telemetry tests will fail (until the browser is restarted):

  Chrome
    ...
    Test 1
      Tracing.start successful
      Test fails without crashing the browser
      Tracing.stop is never called <------------------------+
    Test 2                                                  |
      Tracing.start FAILS because tracing is still running -+
    ...

This can lead to a chain of confusing failures:

  TracingUnexpectedResponseException: Inspector returned unexpected response for Tracing.start:
  {
    "id": 1, 
    "error": {
      "message": "Tracing is already started", 
      "code": -32603
    }
  }

Example: https://uberchromegw.corp.google.com/i/chromium.perf/builders/Android%20Nexus9%20Perf%20%283%29/builds/931 (there are a couple of actual failures in smoothness.key_mobile_sites_smooth.reference and all subsequent tests with device affinity 5 fail).
 
Cc: zh...@chromium.org

Comment 2 by zh...@chromium.org, Mar 17 2016

Cc: eakuefner@chromium.org
Owner: zh...@chromium.org
Status: Assigned (was: Untriaged)
I will take this.

Comment 3 by zh...@chromium.org, Mar 19 2016

Status: Fixed (was: Assigned)
Should be fixed at https://codereview.chromium.org/1818593002/. Reopen if seen otherwise.

Sign in to add a comment