Bug 1791314 Comment 27 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
### Beta/Release Uplift Approval Request
* **User impact if declined**: Security sensitive crash
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: Load the testcase in an Asan build.
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This is a bit risky patch. It changes certain synchronous operations to be asynchronous
* **String changes made/needed**: NA
* **Is Android affected?**: Yes

### ESR Uplift Approval Request
* **If this is not a sec:{high,crit} bug, please state case for ESR consideration**: 
* **User impact if declined**: 
* **Fix Landed on Version**: 
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**:
Revision 1 by
on 
### Beta/Release Uplift Approval Request
* **User impact if declined**: Security sensitive crash
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: Load the testcase in an Asan build.
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This is a bit risky patch. It changes certain synchronous operations to be asynchronous
* **String changes made/needed**: NA
* **Is Android affected?**: Yes

### ESR Uplift Approval Request
* **If this is not a sec:{high,crit} bug, please state case for ESR consideration**: 
* **User impact if declined**: 
* **Fix Landed on Version**: 
* **Risk to taking this patch**: 
* **Why is the change risky/not risky? (and alternatives if risky)**:

Back to Bug 1791314 Comment 27