Closed
Bug 1030523
Opened 10 years ago
Closed 10 years ago
Content view sometimes blank on load
Categories
(Firefox for Android Graveyard :: Toolbar, defect)
Tracking
(firefox34 wontfix, firefox35+ fixed, firefox36+ fixed, fennec+)
RESOLVED
FIXED
Firefox 36
People
(Reporter: jwir3, Assigned: kats)
References
Details
Attachments
(4 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
text/x-vhdl
|
Details | |
(deleted),
patch
|
snorp
:
review+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
STR:
1. Open a new tab (doesn't happen consistently).
Expected results:
Page loads, content is displayed
Actual Results:
Page shows loading bar, favicon comes up, but content area is blank ( see screenshot).
This only happens to me when I am navigating to a URL from an outside source (e.g. Google search or a link from my email). I can see that the content is loaded because on the tabs view, it renders.
Reporter | ||
Comment 1•10 years ago
|
||
Reporter | ||
Comment 2•10 years ago
|
||
Also, any content I view from this point on (e.g. loading a new tab or selecting an existing tab) has the same blank content.
Reporter | ||
Comment 3•10 years ago
|
||
This is on a nexus 5 with stock KitKat (rooted).
Hardware: Other → x86
Version: unspecified → Firefox 31
Updated•10 years ago
|
Hardware: x86 → ARM
Assignee | ||
Comment 4•10 years ago
|
||
I saw some chatter on IRC about this as well, but haven't seen it happen myself. Would like STR so I can repro and debug.
Keywords: steps-wanted
Updated•10 years ago
|
tracking-fennec: --- → ?
Updated•10 years ago
|
Assignee: nobody → snorp
tracking-fennec: ? → 33+
Updated•10 years ago
|
tracking-fennec: 33+ → 34+
Updated•10 years ago
|
tracking-fennec: 34+ → +
Assignee | ||
Comment 6•10 years ago
|
||
I haven't yet run into this in Fennec, but I have seen it in a webapp that I have that runs on aurora. If I start the webapp, then immediately hit the home button to put the webapp in the background (while it is still on the splash screen), and then later bring it back to the foreground, it doesn't render anything and just stays blank the whole time. I suspect this is the same issue - somewhere during startup the GL context is not available and we get stuck in a bad state or something like that.
Assignee | ||
Comment 7•10 years ago
|
||
I grabbed a log using the STR I described above (with which I can reliably reproduce the problem). There is a stacktrace in the log that indicates the allocation of the GL surface is failing, as I surmised. The code is supposed to be able to handle this scenario, by calling updateCompositor() again fennec becomes visible again. A quick glance through the code shows that the change at https://hg.mozilla.org/mozilla-central/rev/038356d89dc2 may have broken this, because surfaceChanged(...) is not called any more when onSizeChanged is invoked and the compositor isn't created yet. That seems like a bug, and I"m pretty sure that patch isn't doing what it was intended to be doing, because the surfaceChanged(...) call does more or less the same thing as the bottom of onSizeChanged, and it doesn't make sense to do both.
I'm spinning up a local build with some extra logging to confirm my theory.
Assignee | ||
Comment 8•10 years ago
|
||
I confirmed my theory with logging and backing out that change improves the situation considerably. With my patch the compositor starts up properly but the screen remains blank initially. Using the app switcher to re-select the webapp makes it render normally again.
Assignee | ||
Comment 9•10 years ago
|
||
This helps quite a bit with the STR I was able to repro, although it doesn't fix it entirely as mentioned above. I'm not sure what I need to do to get it to repaint right away.
Attachment #8501019 -
Flags: review?(snorp)
Comment 10•10 years ago
|
||
I think I can reproduce this consistently when I try to load shoppersdrugmart.ca. It loads an empty page in Firefox 32, 33, and 35 but loads fine in Chrome on my Nexus 5 and Asus Transformer tablet.
Assignee | ||
Comment 11•10 years ago
|
||
I think that's a different issue. This bug is about Firefox rendering *all* pages as blank, while the tab tray shows that the pages definitely have content. shoppersdrugmart.ca is showing up blank for me as well, but it doesn't get Firefox stuck in a bad state. That's worth filing a separate bug about though.
Comment 12•10 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #11)
> That's worth filing a separate bug about though.
Filed bug 1080743, thanks kats.
Comment 13•10 years ago
|
||
Comment on attachment 8501019 [details] [diff] [review]
Backout of earlier cset
Review of attachment 8501019 [details] [diff] [review]:
-----------------------------------------------------------------
Still not sure I fully understand what is going on here, but if this makes things better then I'm happy. I'd like to talk about this some more this week to see if we can get the last issue figured out.
Attachment #8501019 -
Flags: review?(snorp) → review+
Assignee | ||
Comment 14•10 years ago
|
||
Assignee: snorp → bugmail.mozilla
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Updated•10 years ago
|
Comment 16•10 years ago
|
||
[Tracking Requested - why for this release]: resolves a known issue
status-firefox34:
--- → wontfix
status-firefox35:
--- → affected
status-firefox36:
--- → affected
tracking-firefox35:
--- → ?
tracking-firefox36:
--- → ?
Updated•10 years ago
|
Assignee | ||
Comment 17•10 years ago
|
||
Comment on attachment 8501019 [details] [diff] [review]
Backout of earlier cset
Approval Request Comment
[Feature/regressing bug #]: bug 834243
[User impact if declined]: sometimes the content view ends up blank in fennec, which is very annoying. once in this state you pretty much have to restart fennec to fix it
[Describe test coverage new/current, TBPL]: on m-c for a while now
[Risks and why]: affects android only. the code being modified is obviously wrong so I would say there's a low risk that this will make things worse
[String/UUID change made/needed]: none
Attachment #8501019 -
Flags: approval-mozilla-beta?
Assignee | ||
Updated•10 years ago
|
Keywords: steps-wanted
Updated•10 years ago
|
Attachment #8501019 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 18•10 years ago
|
||
Comment 20•5 years ago
|
||
Occurs for me in Firefox 68, Android 8.0.0, Moto G6 (as of July through August 2019)
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•