Bug 1873019 Comment 22 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**: Users who refresh about:welcome on 2nd slide onward and then try to navigate back/forth will result in an unresponsive browser.
* **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**: Steps can be found in the [phabricator patch](https://phabricator.services.mozilla.com/D198302)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Reverted a change from 4 months ago which was a useEffect dependency changes in LanguageSwitcher in [1849882](https://bugzilla.mozilla.org/show_bug.cgi?id=1849882) causing the regression
* **String changes made/needed**: 
* **Is Android affected?**: No
Requesting uplift only on the [beta patch](https://phabricator.services.mozilla.com/D198315) to avoid conflicts with [D197884](https://phabricator.services.mozilla.com/D197884).

### Beta/Release Uplift Approval Request
* **User impact if declined**: Users who refresh about:welcome on 2nd slide onward and then try to navigate back/forth will result in an unresponsive browser.
* **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**: Steps can be found in the [phabricator patch](https://phabricator.services.mozilla.com/D198302)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Reverted a change from 4 months ago which was a useEffect dependency changes in LanguageSwitcher in [1849882](https://bugzilla.mozilla.org/show_bug.cgi?id=1849882) causing the regression
* **String changes made/needed**: 
* **Is Android affected?**: No
Requesting uplift on the [beta patch](https://phabricator.services.mozilla.com/D198315) to avoid conflicts with [D197884](https://phabricator.services.mozilla.com/D197884).

### Beta/Release Uplift Approval Request
* **User impact if declined**: Users who refresh about:welcome on 2nd slide onward and then try to navigate back/forth will result in an unresponsive browser.
* **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**: Steps can be found in the [phabricator patch](https://phabricator.services.mozilla.com/D198302)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Reverted a change from 4 months ago which was a useEffect dependency changes in LanguageSwitcher in [1849882](https://bugzilla.mozilla.org/show_bug.cgi?id=1849882) causing the regression
* **String changes made/needed**: 
* **Is Android affected?**: No
Requesting uplift on the [beta patch](https://phabricator.services.mozilla.com/D198315) to avoid conflicts with [D197884](https://phabricator.services.mozilla.com/D197884).

### Beta/Release Uplift Approval Request
* **User impact if declined**: Users who refresh about:welcome on 2nd slide onward and then try to navigate back/forth will result in an unresponsive browser.
* **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**: Steps can be found in the [phabricator patch](https://phabricator.services.mozilla.com/D198302)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Reverted a change from 4 months ago which was a useEffect dependency change in LanguageSwitcher in [1849882](https://bugzilla.mozilla.org/show_bug.cgi?id=1849882) causing the initial regression
* **String changes made/needed**: 
* **Is Android affected?**: No

Back to Bug 1873019 Comment 22