Skip to content

Commit 666a967

Browse files
AlinaVarkkiDevtools-frontend LUCI CQ
authored andcommitted
Reenable large CPU profile load test
It was throwing "no insight" error even though insights are null in CPU profiles Bug: 433466849 Change-Id: I91247777c6c1f170ff2ea0aa1ef0e131aa42030e Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/7013752 Auto-Submit: Alina Varkki <[email protected]> Reviewed-by: Connor Clark <[email protected]> Commit-Queue: Connor Clark <[email protected]> Reviewed-by: Jack Franklin <[email protected]>
1 parent ba11bff commit 666a967

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

test/perf/panels/performance/trace-load_test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,7 @@ async function timeFixture(fixture: string): Promise<number> {
4949

5050
describe('Performance panel trace load performance', () => {
5151
const allTestValues: Array<{name: string, values: number[]}> = [];
52-
// Skipped because this throws range errors
53-
describe.skip('[crbug.com/433466849] Large CPU profile load benchmark', () => {
52+
describe('Large CPU profile load benchmark', () => {
5453
beforeEach(async () => {
5554
// Reload devtools to get a fresh version of the panel on each
5655
// run and prevent a skew due to caching, etc.

0 commit comments

Comments
 (0)