1754487 - [android 5] Pages are zoomed when changing from portrait to landscape and viceversa

Status: NEW

(GeckoView :: General, defect, P3)

Description

[Petru-Mugurel Lingurar [:petru]]

From github: https://github.com/mozilla-mobile/focus-android/issues/6385.

Steps to reproduce

1. Open a page, like wikipedia.org.
2. Turn the device to landscape.
3. Turn it to portrait.
4. Repeat steps 2 & 3 and observe the page.

Expected behavior

The page is displayed without issues.

Actual behavior

The page is zoomed.

Device information

• Android device: Samsung Galaxy Tab A6 (Android 5.1.1), Xiaomi mi4i (Android 5.0.2)
• Not reproducible on Samsung Galaxy Note 8 (Android 9), nor Lenovo Tab M10 (Android 10)
• Focus version: Focus Nightly from 2/8, Beta 98.0.0-beta.1
• Please see the similar issue filed on Fenix: 17856

https://user-images.githubusercontent.com/40694158/153171524-d9b7b4d0-6f5c-4b09-9ba8-bba1478d4e9f.mp4

Change performed by the Move to Bugzilla add-on.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jwatt, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Jonathan Watt [:jwatt]]

Botond, I guess this is APZ related and you may be in a better position to set severity.

Comment 3

[Botond Ballo [:botond] [away until Jun 29]]

Yeah, how the zoom level changes during an orientation change is generally governed by MobileViewportManager.

I do recall it being an explicit design intent that changing orientation and back should preserve the zoom level, so I would definitely consider this a bug.

Severity: S3. While this is mildly annoying, it's easy to work around by zooming back out after the second orientation change.

Note, I can't reproduce the issue on a Galaxy S7. Based on the bug report, it may only affect tablets, and I don't have access to one at the moment to test there.

Comment 4

[Hiroyuki Ikezoe (:hiro)]

(In reply to Petru-Mugurel Lingurar [:petru] from comment #0)

Device information

• Android device: Samsung Galaxy Tab A6 (Android 5.1.1), Xiaomi mi4i (Android 5.0.2)
• Not reproducible on Samsung Galaxy Note 8 (Android 9), nor Lenovo Tab M10 (Android 10)
• Focus version: Focus Nightly from 2/8, Beta 98.0.0-beta.1
• Please see the similar issue filed on Fenix: 17856

Given that the issue happens on Android 5, doesn't happen on Android 9. Is it likely GeckoView issues, isn't it?

Comment 5

[Hiroyuki Ikezoe (:hiro)]

Or there's a race that the notification of orientation changes gets delayed on the device (or the Android version).

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

The product::component has been changed since the backlog priority was decided, so we're resetting it.
For more information, please visit auto_nag documentation.

Comment 7

[Petru-Mugurel Lingurar [:petru]]

(In reply to Hiroyuki Ikezoe (:hiro) from comment #4)

(In reply to Petru-Mugurel Lingurar [:petru] from comment #0)
Given that the issue happens on Android 5, doesn't happen on Android 9. Is it likely GeckoView issues, isn't it?

Looking also at https://github.com/mozilla-mobile/fenix/issues/17856 it seems like this is an intermittent issue, possibly caused by a race.

Comment 8

[Chris Peterson [:cpeterson]]

Hiro, does this bug belong in the Layout or APZ component? Is this a bug in GeckoView or in Gecko?

Comment 9

[Hiroyuki Ikezoe (:hiro)]

It's hard to tell without diagnose on the devices where the issue can be reproducible.

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:amoya, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 11

[Mira Lobontiu (Android QA)]

Still reproducible on Xiaomi mi4i (Android 5.0.2) on the latest Focus nightly 106.0a1 from 9/2, and the latest Beta 105.0b4.