### Beta/Release Uplift Approval Request * **User impact if declined**: Integrating this patch could potentially avoid 1000~2000 main process crashes in 111.0.2 (if a 111.0.2 is planned). * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This is a small, low-impact change that has now lived for some time in Nightly and Beta without introducing any problems there (to the best of my knowledge). This looks like an easy win to prevent some users from crashing if a 111.0.2 is planned; although it's not a huge problem either to let them wait for 112.0. * **String changes made/needed**: * **Is Android affected?**: No
Bug 1733532 Comment 10 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**: Integrating this patch could potentially avoid 1000~2000 main process crashes in 111.0.2 (if a 111.0.2 is planned). * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No (the users that crash are all using Release builds) * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This is a small, low-impact change that has now lived for some time in Nightly and Beta without introducing any problems there (to the best of my knowledge). This looks like an easy win to prevent some users from crashing if a 111.0.2 is planned; although it's not a huge problem either to let them wait for 112.0. * **String changes made/needed**: * **Is Android affected?**: No
### Beta/Release Uplift Approval Request * **User impact if declined**: Integrating this patch could potentially avoid 1000~2000 main process crashes in 111.0.2 (if a 111.0.2 is planned). * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No (mostly Release users, a few Beta users) * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This is a small, low-impact change that has now lived for some time in Nightly and Beta without introducing any problems there (to the best of my knowledge). This looks like an easy win to prevent some users from crashing if a 111.0.2 is planned; although it's not a huge problem either to let them wait for 112.0. We've always had a few crashes in Beta (see below) and so far we received none in 112 beta versions, suggesting that the patch works. * **String changes made/needed**: * **Is Android affected?**: No [Crashes in beta versions sorted by Version](https://crash-stats.mozilla.org/search/?release_channel=beta&release_channel=aurora&signature=%3Dmozilla%3A%3Afreestanding%3A%3Apatched_NtMapViewOfSection%20%7C%20Thread32Next&product=Firefox&date=%3E%3D2022-09-24T08%3A28%3A00.000Z&date=%3C2023-03-24T08%3A28%3A00.000Z&_facets=url&_facets=user_comments&_facets=install_time&_facets=version&_facets=address&_facets=moz_crash_reason&_facets=reason&_facets=build_id&_facets=platform_pretty_version&_facets=signature&_facets=useragent_locale&_sort=-date&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-version): 5 111.0rc2 8 5.56 % 3 111.0rc1 13 9.03 % 8 111.0b5 6 4.17 % 19 111.0b4 3 2.08 % 30 111.0b3 1 0.69 % 2 110.0rc1 15 10.42 % 4 110.0b9 12 8.33 % 18 110.0b7 3 2.08 % 29 110.0b6 1 0.69 % 15 110.0b5 4 2.78 % 14 110.0b2 4 2.78 % [...]
### Beta/Release Uplift Approval Request * **User impact if declined**: Integrating this patch could potentially avoid 1000~2000 main process crashes in 111.0.2 (if a 111.0.2 is planned). * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No. The users are mostly using Release channel except for a few Beta users. We've always had a few crashes in Beta (see below) and so far we received none in 112 beta versions, suggesting that the patch works. * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This is a small, low-impact change that has now lived for some time in Nightly and Beta without introducing any problems there (to the best of my knowledge). This looks like an easy win to prevent some users from crashing if a 111.0.2 is planned; although it's not a huge problem either to let them wait for 112.0. * **String changes made/needed**: * **Is Android affected?**: No [Crashes in beta versions sorted by Version](https://crash-stats.mozilla.org/search/?release_channel=beta&release_channel=aurora&signature=%3Dmozilla%3A%3Afreestanding%3A%3Apatched_NtMapViewOfSection%20%7C%20Thread32Next&product=Firefox&date=%3E%3D2022-09-24T08%3A28%3A00.000Z&date=%3C2023-03-24T08%3A28%3A00.000Z&_facets=url&_facets=user_comments&_facets=install_time&_facets=version&_facets=address&_facets=moz_crash_reason&_facets=reason&_facets=build_id&_facets=platform_pretty_version&_facets=signature&_facets=useragent_locale&_sort=-date&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-version): 5 111.0rc2 8 5.56 % 3 111.0rc1 13 9.03 % 8 111.0b5 6 4.17 % 19 111.0b4 3 2.08 % 30 111.0b3 1 0.69 % 2 110.0rc1 15 10.42 % 4 110.0b9 12 8.33 % 18 110.0b7 3 2.08 % 29 110.0b6 1 0.69 % 15 110.0b5 4 2.78 % 14 110.0b2 4 2.78 % [...]
### Beta/Release Uplift Approval Request * **User impact if declined**: Integrating this patch could potentially avoid 1000~2000 main process crashes in 111.0.2 (if a 111.0.2 is planned). * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No. Not verified but it has lived there. The users are mostly using Release channel except for a few Beta users. We've always had a few crashes in Beta (see below) and so far we received none in 112 beta versions, suggesting that the patch works. * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This is a small, low-impact change that has now lived for some time in Nightly and Beta without introducing any problems there (to the best of my knowledge). This looks like an easy win to prevent some users from crashing if a 111.0.2 is planned; although it's not a huge problem either to let them wait for 112.0. * **String changes made/needed**: * **Is Android affected?**: No [Crashes in beta versions sorted by Version](https://crash-stats.mozilla.org/search/?release_channel=beta&release_channel=aurora&signature=%3Dmozilla%3A%3Afreestanding%3A%3Apatched_NtMapViewOfSection%20%7C%20Thread32Next&product=Firefox&date=%3E%3D2022-09-24T08%3A28%3A00.000Z&date=%3C2023-03-24T08%3A28%3A00.000Z&_facets=url&_facets=user_comments&_facets=install_time&_facets=version&_facets=address&_facets=moz_crash_reason&_facets=reason&_facets=build_id&_facets=platform_pretty_version&_facets=signature&_facets=useragent_locale&_sort=-date&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-version): 5 111.0rc2 8 5.56 % 3 111.0rc1 13 9.03 % 8 111.0b5 6 4.17 % 19 111.0b4 3 2.08 % 30 111.0b3 1 0.69 % 2 110.0rc1 15 10.42 % 4 110.0b9 12 8.33 % 18 110.0b7 3 2.08 % 29 110.0b6 1 0.69 % 15 110.0b5 4 2.78 % 14 110.0b2 4 2.78 % [...]
### Beta/Release Uplift Approval Request * **User impact if declined**: Integrating this patch could potentially avoid 1000~2000 main process crashes in 111.0.2 (if a 111.0.2 is planned). * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: Yes (in Beta). The users are mostly using Release channel except for a few Beta users. We've always had a few crashes in Beta (see below) and so far we received none in 112 beta versions, suggesting that the patch works. * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This is a small, low-impact change that has now lived for some time in Nightly and Beta without introducing any problems there (to the best of my knowledge). This looks like an easy win to prevent some users from crashing if a 111.0.2 is planned; although it's not a huge problem either to let them wait for 112.0. * **String changes made/needed**: * **Is Android affected?**: No [Crashes in beta versions sorted by Version](https://crash-stats.mozilla.org/search/?release_channel=beta&release_channel=aurora&signature=%3Dmozilla%3A%3Afreestanding%3A%3Apatched_NtMapViewOfSection%20%7C%20Thread32Next&product=Firefox&date=%3E%3D2022-09-24T08%3A28%3A00.000Z&date=%3C2023-03-24T08%3A28%3A00.000Z&_facets=url&_facets=user_comments&_facets=install_time&_facets=version&_facets=address&_facets=moz_crash_reason&_facets=reason&_facets=build_id&_facets=platform_pretty_version&_facets=signature&_facets=useragent_locale&_sort=-date&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-version): 5 111.0rc2 8 5.56 % 3 111.0rc1 13 9.03 % 8 111.0b5 6 4.17 % 19 111.0b4 3 2.08 % 30 111.0b3 1 0.69 % 2 110.0rc1 15 10.42 % 4 110.0b9 12 8.33 % 18 110.0b7 3 2.08 % 29 110.0b6 1 0.69 % 15 110.0b5 4 2.78 % 14 110.0b2 4 2.78 % [...]