Closed Bug 1730504 Opened 1 year ago Closed 1 year ago

Refactor the OOM testing framework a little


(Core :: JavaScript Engine, task, P3)




94 Branch
Tracking Status
firefox94 --- fixed


(Reporter: jonco, Assigned: jonco)


(Blocks 1 open bug)



(5 files)

Currently the OOM testing framework is implemented by one large function, RunIterativeFailureTest. This could be split up to make it easier to understand.

This adds an IterativeFailureTest class which encompasses the current testing
framework with the testing code in a 'test' method.

This splits the method up into several sub-methods to make it easier to see
what's going on.

Depends on D125400

It's possible to compile OOM testing facilites outside of debug builds with the
--enable-oom-breakpoint flag so the test should work in that case too.

Depends on D125401

Pushed by
Refactor iterative failure test framework into a class r=tcampbell
Split up IterativeFailureTest::test method into separate methods r=tcampbell
Make OOM tests work outside debug builds r=tcampbell
Make ParseIterativeFailureTestParams into a method on IterativeFailureTest r=tcampbell
Remove IterativeFailureTest::Params class and store fields directly in the class r=tcampbell
You need to log in before you can comment on or make changes to this bug.