Bug 1551913 Comment 23 Edit History

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

### Beta/Release Uplift Approval Request
* **User impact if declined**: This security bug and Bug 1552206 would not be resolved for an additional train cycle.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This patch involves low-level OS-specific code. Also, the patch from Bug 1570396 has not had very much time to "bake" on Nightly. Seems like the only real alternative here is to wait and possibly uplift to release later.
* **String changes made/needed**:
### Beta/Release Uplift Approval Request
* **User impact if declined**: This security bug and Bug 1552206 would not be resolved for an additional train cycle.
* **Is this code covered by automated tests?**: Yes, but they aren't merged yet.
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This patch involves low-level OS-specific code. Also, the patch from Bug 1570396 has not had very much time to "bake" on Nightly. Seems like the only real alternative here is to wait and possibly uplift to release later.
* **String changes made/needed**:
### Beta/Release Uplift Approval Request
* **User impact if declined**: This security bug and Bug 1552206 would not be resolved for an additional train cycle.
* **Is this code covered by automated tests?**: Yes, but they aren't merged yet (patch is attached to this bug).
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This patch involves low-level OS-specific code. Also, the patch from Bug 1570396 has not had very much time to "bake" on Nightly. Seems like the only real alternative here is to wait and possibly uplift to release later.
* **String changes made/needed**:

Back to Bug 1551913 Comment 23