Closed Bug 1762476 Opened 2 years ago Closed 2 years ago

Verify promise before calling AppendNativeHandler in L10nMutations

Categories

(Core :: Internationalization: Localization, defect, P2)

Product:
Core
Component:
Internationalization: Localization
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
100 Branch
Tracking Flags:
Tracking Status
firefox100 --- fixed

People

(Reporter: nordzilla, Assigned: nordzilla)

References

Details

Attachments

(1 file)

Bug 1762476 - Validate promise existence in L10nMutations r=dminor
48 bytes, text/x-phabricator-request
Details | Review

Bug 1739143 introduced code that invokes AppendNativeHandler on a promise in L10nMutations.cpp.

We need to ensure that the promise exists before invoking a method on it.

This may fix the crash in Bug 1751836

Validates that the TranslateElements promise exists before
attempting to append a native handler to it.

Pushed by enordin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6efe81e61617
Validate promise existence in L10nMutations r=dminor

https://hg.mozilla.org/mozilla-central/rev/6efe81e61617

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox100: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.