Closed Bug 1210760 Opened 9 years ago Closed 9 years ago

Don't simulate OOM in ExceptionHandlerBailout()

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

Following on from bug 1171909 and bug 1201057, update ExceptionHandlerBailout() so that we don't simulate OOM while bailing out
BailoutIonToBaseline() is called elsewhere where we do simulate OOM so there should be some coverage of this.

This change is motivated by the unhandlable OOM crash in ExceptionHandlerBailout() happening when testing other OOM handling patches.
Patch to add AutoEnterOOMUnsafeRegion around where we call ExceptionHandlerBailout().
Attachment #8669032 - Flags: review?(terrence)
Attachment #8669032 - Flags: review?(terrence) → review+
https://hg.mozilla.org/mozilla-central/rev/5118292ae7f3
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: