### Beta/Release Uplift Approval Request * **User impact if declined**: NOTE: The release request should probably be conditional on confirming that bug 1668113 is fixed by this. The beta request probably still stands, this fixes a regression from bug 1581876. * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: Yes * **If yes, steps to reproduce**: If possible, test bug 1668113, otherwise no test needed, if the automated test passes. * **List of other uplifts needed**: none * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Fixes an assumption in a layout API that the caller in bug 1581876 broke. * **String changes made/needed**: none
Bug 1665447 Comment 5 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**: NOTE: The release request should probably be conditional on confirming that bug 1668113 is fixed by this. The beta request probably still stands, this fixes a regression from bug 1581876. * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: Yes * **If yes, steps to reproduce**: If possible, test bug 1668113, otherwise no test needed, if the automated test passes. * **List of other uplifts needed**: none * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Simple fix for an assumption in a layout API that the caller in bug 1581876 broke. * **String changes made/needed**: none