Bug 1479433 Comment 14 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**: Firefox is a little slower
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: Bug 1532746
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The change only impacts CSS for the containers and has been verified in Nightly as working for 9 days.
* **String changes made/needed**: N/A
### Beta/Release Uplift Approval Request
* **User impact if declined**: Firefox is a little slower
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: Bug 1532746
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The change only impacts CSS for the containers and has been verified in Nightly as working for 9 days.
* **String changes made/needed**: N/A, Bug 1532746 has two container strings in it though "containers-icon-fence.label" and "containers-color-toolbar.label"
### Beta/Release Uplift Approval Request
* **User impact if declined**: Firefox is a little slower
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: Bug 1532746
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The change only impacts CSS for the containers and has been verified in Nightly as working for 9 days. The changes in Bug 1532746 just add an icon but they depend on this CSS change. The impact to not landing that is the facebook container addon won't be able to use the correct icon.
* **String changes made/needed**: N/A, Bug 1532746 has two container strings in it though "containers-icon-fence.label" and "containers-color-toolbar.label"

Back to Bug 1479433 Comment 14