Bug 1977303 Comment 20 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/Reason for urgency**: Crashes in beta.
* **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**: 1. Using a device on Android 6 (API 23), click on the address or create a new tab from the tabs tray.
2. Using a device on Android 12 (API 32), click on the address or create a new tab from the tabs tray.
3. In step 1, there should be text focus on the search bar and the app should not crash.
4. In step 2, there should be text focus on the search bar and the search bar position does not move.
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This change does not have UI test coverage because it's for an API level we will no longer be supporting soon. Manual QA verification to notice any visual mistakes is recommended.
* **String changes made/needed**: n/a
* **Is Android affected?**: Yes
Revision 1 by
on 
(I tried using the new Lando uplift request and ended up with multiple patches and no approval request form - sorry!)

### Beta/Release Uplift Approval Request
* **User impact if declined/Reason for urgency**: Crashes in beta.
* **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**: 1. Using a device on Android 6 (API 23), click on the address or create a new tab from the tabs tray.
2. Using a device on Android 12 (API 32), click on the address or create a new tab from the tabs tray.
3. In step 1, there should be text focus on the search bar and the app should not crash.
4. In step 2, there should be text focus on the search bar and the search bar position does not move.
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: This change does not have UI test coverage because it's for an API level we will no longer be supporting soon. Manual QA verification to notice any visual mistakes is recommended.
* **String changes made/needed**: n/a
* **Is Android affected?**: Yes

Back to Bug 1977303 Comment 20