Skip to content

Commit

Permalink
Re-enable disabled test in PerfBenchmarkWithProfilingTest now that a
Browse files Browse the repository at this point in the history
new version of Telemetry (with the relevant API change) has rolled into
chromium.

The API change is at https://crrev.com/c/3784382.

Change-Id: Ie3f92be846cd9a38119be5c38edcb5da48214975
Bug: chromium:1338581
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3793788
Reviewed-by: Mikhail Khokhlov <khokhlov@google.com>
Commit-Queue: Tushar Agarwal <agarwaltushar@google.com>
Cr-Commit-Position: refs/heads/main@{#1030205}
  • Loading branch information
Tushar Agarwal authored and Chromium LUCI CQ committed Aug 1, 2022
1 parent ae48fc2 commit 101d14b
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@ def testWithoutAndroidBrowser(self):
text_config = options.config.system_trace_config.GetTextConfig()
self.assertNotIn('name: "linux.perf"', text_config)

# TODO(http://crbug/1338581): Re-enable this test.
@unittest.skip("Temporary disabled to facilitate an API change in Telemetry.")
def testWithAndroidBrowser(self):
benchmark = PerfBenchmarkForTesting()
Expand Down

0 comments on commit 101d14b

Please sign in to comment.