1193753 - Exiting Settings menu does not repaint the page; a black page is displayed

Status: RESOLVED DUPLICATE of bug 1197176

(Firefox for Android Graveyard :: Toolbar, defect)

Description

[Flaviu Cos]

Environment: 
Device: Asus Transformer Pad (Android 4.2.1);
Build: Nightly 43.0a1 (2015-08-11);

Steps to reproduce:
1. Load a page (e.g.: bugzilla.mozilla.org);
2. Go to Settings;
3. Tap on back button.

Expected result:
The Settings page is dismissed and page loaded at step 1 is displayed.

Actual result:
A black page is displayed.
Tapping or panning on the black area will display the page loaded on step 1.

Notes:
Please check the video:
https://youtu.be/HqjcfyywNuA

Comment 1

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Snorp, do you know what's going on here?

Comment 2

[James Willcox (:snorp) ([e-mail]) (he/him)]

We pause the compositor when we leave the main activity, then resume it when we come back. I would say something is going wrong with that latter part. Flaviu, can you capture logcat for us? Does this happen every time, or just sometimes? Is it a regression?

Comment 3

[Flaviu Cos]

Attachment: logs.txt

This issue is a regression.
Regression window:
Last good build: 2015-07-07
First bad build: 2015-07-08
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=e7e69cc8c07b&tochange=9b902b7669ae

Attached are the logs from logcat.

Comment 4

[Flaviu Cos]

Mozilla-inbound regression range:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=530ecc275dd9&tochange=004e123383ae

Comment 5

[Kevin Brosnan [Ex-Mozilla]]

[Tracking Requested - why for this release]: regression

Roc looks like 1143575 caused this.

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 42+ since this is a recent regression.

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I don't see this on my Nexus S running Android 4.1.2 and a debug build of 1eb1c6f76ffc (which includes the fix for bug 1197176). Is it still happening with that build on your device/Android version? Is it 100% reproducible with the steps in comment #0? Can you narrow down the device/Android version(s) required to see this bug?

Comment 8

[Flaviu Cos]

(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #7)
> I don't see this on my Nexus S running Android 4.1.2 and a debug build of
> 1eb1c6f76ffc (which includes the fix for bug 1197176). Is it still happening
> with that build on your device/Android version? 

Tested on latest nightly (2015-08-26) and the issue is no longer reproducible. 
The black screen appears for a fraction of a second before the page is displayed.

> Is it 100% reproducible with the steps in comment #0? Can you narrow down the device/Android version(s)
> required to see this bug?

Tested on multiple devices:

Asus Transformer Pad (Android 4.2.1) -> Reproducible;
Asus Transformer Pad (Android 4.0.3) -> Reproducible
Samsung Galaxy Tab 2 10" (Android 4.0.4) -> Reproducible;
Nexus 9 (Android 5.1.1) -> Not reproducible;
Motorola Razr (Android 4.4.4) -> Not reproducible.

Comment 9

[Kartikaya Gupta (email:[e-mail])]

Do you happen to have the "Don't keep activities" option under the Android Developer Options set?

Comment 10

[Flaviu Cos]

"Don't keep activities" option is not enabled on any device tested on comment 8.

Comment 11

[Kartikaya Gupta (email:[e-mail])]

Hm, ok. I guess if this was fixed on Nightly by bug 1197176 we can uplift that to Aurora, it's pretty safe. We should probably confirm that's what fixed it though, by getting a reverse regression window.

Comment 12

[James Willcox (:snorp) ([e-mail]) (he/him)]

Kats, can you go ahead with the uplift request? Thanks!

Comment 13

[Kartikaya Gupta (email:[e-mail])]

Done

Comment 14

[Liz Henry (:lizzard) (relman/hg->git project)]

Flaviu, could you verify that this is fixed for 42 and 43?

Comment 15

[Flaviu Cos]

This issue is no longer reproducible on Nightly 43 (2015-09-01) and Aurora 42 (2015-09-01).
Tested on Asus Transformer Pad (Android 4.2.1).

Comment 16

[Kartikaya Gupta (email:[e-mail])]

Great, thanks. Assuming fixed by uplift of bug 1197176.

Comment 18

[Stefan Deiac]

Based on latest update of resolution, I will remove the qe-verify flag, thanks.