Closed Bug 1110229 Opened 9 years ago Closed 9 years ago

Regression: vertical bars painted in tiles on screen when scrolling

Categories

(Firefox for Android Graveyard :: Toolbar, defect)

37 Branch
ARM
Android
defect
Not set
major

Tracking

(firefox34 unaffected, firefox35+ verified, firefox36+ verified, firefox37+ verified, fennec35+)

VERIFIED FIXED
Firefox 37
Tracking Status
firefox34 --- unaffected
firefox35 + verified
firefox36 + verified
firefox37 + verified
fennec 35+ ---

People

(Reporter: aaronmt, Assigned: BenWa)

References

()

Details

(Keywords: regression, reproducible, testcase)

Attachments

(1 file)

Currently on Nightly (12/11), Aurora (12/11), Beta (mozilla-beta 35), when scrolling through content rapidly one will see vertical bars painted on-screen (likely depending on content).

See video: http://youtu.be/wwfoMz_0pvY

Using the e.g, URL to demonstrate this behaviour 

  http://ihackernews.com/comments/8730903
  
mozilla-inbound pushlog

  https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=97068dca449e&tochange=81eafec98b48

mwoodrow@mozilla.com
Wed Oct 08 00:13:22 2014 +0000	81eafec98b48	Matt Woodrow — Bug 1077842 - Simplify invalid regions for tiled painted layers to avoid complex region operations. r=jrmuizel

43c963b56723	Matt Woodrow — Bug 1078109 - Make sure we initialize tiled textures to the format we're going to upload with. r=jgilbert

--
Nightly (12/11)
LG Nexus 5 (Android 5.0)
Flags: in-testsuite?
Assignee: nobody → matt.woodrow
Status: NEW → ASSIGNED
Is this Android L only, do we know?
(In reply to Milan Sreckovic [:milan] from comment #1)
> Is this Android L only, do we know?

No. I can reproduce this on Android 4.x as well (e.g, my HTC One, S4/S5)
This sounds like bug 1073554.
Severity: normal → major
Keywords: testcase
tracking-fennec: ? → 35+
I don't think we can ship Firefox 35 with this bug, it's embarrassing.
Jeff, do you have a device where this reproduces?
Flags: needinfo?(jgilbert)
I tried a build with matt's patches mentioned in comemnt #0 backed out. Before the backout I saw both horizontal and vertical dark lines when scrolling quickly. With the backout I only see the horizontal ones.
Attached patch patchSplinter Review
This patch fixes the problem for me. A small tweak to how we do progressive painting. I don't feel very excited to put this on beta however without specific QA resources for this.
Assignee: matt.woodrow → bgirard
Flags: needinfo?(jgilbert)
Attachment #8545412 - Flags: review?(nical.bugzilla)
Comment on attachment 8545412 [details] [diff] [review]
patch

Review of attachment 8545412 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/layers/client/TiledContentClient.cpp
@@ +1508,5 @@
>      if (!aRegionToPaint.IsEmpty()) {
> +      if (mResolution != CSSToParentLayerScale(1)) {
> +        // Paint the entire tile for low-res. This is aimed to fixing low-res resampling
> +        // and to avoid doing costly region accurate painting for a small area.
> +        aRegionToPaint = tileBounds;

We might want to insert with the visible region instead of the thebes callback doesn't allow us to specific a region larger than the layer.
See Also: → 1073554
[Tracking Requested - why for this release]:

(In reply to Aaron Train [:aaronmt] from comment #4)
> I don't think we can ship Firefox 35 with this bug, it's embarrassing.

Given that, it should be tracking.
Attachment #8545412 - Flags: review?(nical.bugzilla) → review+
Comment on attachment 8545412 [details] [diff] [review]
patch

We need this in asap because of bug 1073554 so we're going ahead with uplift based on QE manual testing and also that we're going to do a b11 build to ship by EOD tomorrow so we can collect more data.
Attachment #8545412 - Flags: approval-mozilla-release+
Attachment #8545412 - Flags: approval-mozilla-beta+
Attachment #8545412 - Flags: approval-mozilla-aurora?
Attachment #8545412 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Thanks for the help here. It's appreciated.
Verified as fixed in Firefox 35 beta 11;
Device: Samsung Galaxy S4 (Android 4.4.2).
Verified as fixed builds:
- 37.0a1 2015-01-09;
- 36.0a2 2015-01-09;

Device: Samsung Galaxy S4 (Android 4.4.2).
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.