Closed
Bug 1051134
Opened 10 years ago
Closed 10 years ago
Eideticker regression in checkerboarding metric for cnn / imgur between Thurs Aug 7 and Fri Aug 8
Categories
(Firefox for Android Graveyard :: Toolbar, defect)
Tracking
(firefox34+ fixed, firefox35 fixed, fennec34+)
RESOLVED
FIXED
Firefox 34
People
(Reporter: wlach, Assigned: mattwoodrow)
References
Details
(Whiteboard: [eideticker_regression])
Attachments
(1 file)
4.45 KB,
patch
|
bas.schouten
:
review+
|
Details | Diff | Splinter Review |
So Eideticker is now producing results again regularly for the Galaxy Nexus, and we've already found a regression: http://eideticker.mozilla.org/#/samsung-gn/mozilla-central/cnn/checkerboard/7 http://eideticker.mozilla.org/#/samsung-gn/mozilla-central/imgur/checkerboard/7 Pushlog of range: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=18f408a5984e&tochange=5e74101ce59e There were some graphics commits by :mattwoodrow and :tn for this range, needinfo'ing them.
Flags: needinfo?(tnikkel)
Flags: needinfo?(matt.woodrow)
Comment 1•10 years ago
|
||
Any way we can narrow down the range? My change only changes the offset of a visible rect (not the size), so I would hope that it didn't have any effect on perf.
Flags: needinfo?(tnikkel)
Reporter | ||
Comment 2•10 years ago
|
||
Yeah I can try to narrow down the regression a bit today.
Reporter | ||
Comment 3•10 years ago
|
||
Used mozregression to isolate things a bit more, isolated it to these commits: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=23ea95cbd92f&tochange=dbd45a6fec49 (which means that :tn is off the hook, Matt could you look into this when you get a chance?)
Reporter | ||
Updated•10 years ago
|
tracking-fennec: --- → ?
Updated•10 years ago
|
Assignee: nobody → matt.woodrow
tracking-fennec: ? → 34+
Assignee | ||
Comment 4•10 years ago
|
||
What exactly do these tests measure? And how would I go about reproducing the issues locally, without the camera setup? Scrolling on imgur.com seems entirely dependent on image loading, I can't get it to checkerboard at all.
Flags: needinfo?(matt.woodrow)
Updated•10 years ago
|
Whiteboard: [eideticker_regression]
Reporter | ||
Comment 5•10 years ago
|
||
(In reply to Matt Woodrow (:mattwoodrow) from comment #4) > What exactly do these tests measure? And how would I go about reproducing > the issues locally, without the camera setup? They measure the amount of time where the display was "purple" (i.e. we are waiting for the viewport to redraw) or as we say on the dashboard: "The measure is the sum of the percentages of frames that are checkerboarded over the entire capture. Lower values are better." Do you have a galaxy nexus? You can reproduce the test environment by running eideticker in no capture mode (and optionally gather similar checkerboarding information using internal metrics). https://github.com/wlach/eideticker/blob/master/README.md#console-mode Would a gecko profile be helpful? I could probably gather those without too much effort (we're actually going to be generating eideticker runs with profiling enabled again soon). > Scrolling on imgur.com seems entirely dependent on image loading, I can't > get it to checkerboard at all. I suspect this is highly device dependent.
Flags: needinfo?(matt.woodrow)
Reporter | ||
Comment 6•10 years ago
|
||
(In reply to William Lachance (:wlach) from comment #5) > https://github.com/wlach/eideticker/blob/master/README.md#console-mode Sorry, this should be: https://github.com/mozilla/eideticker/blob/master/README.md#console-mode
Assignee | ||
Comment 7•10 years ago
|
||
I don't have a galaxy nexus, will it work on other devices? Profiles would be very helpful!
Flags: needinfo?(matt.woodrow)
Reporter | ||
Comment 8•10 years ago
|
||
Sorry for the late response... (In reply to Matt Woodrow (:mattwoodrow) from comment #7) > I don't have a galaxy nexus, will it work on other devices? In theory yes, though I'm not sure how reproducible the problem would be. If it would be helpful I can give you access to one of the eideticker machines with a device connected. > Profiles would be very helpful! Before (nightly from 2014-08-07): http://people.mozilla.org/~bgirard/cleopatra/?zippedProfile=http://people.mozilla.org/~wlachance/sps-profile-1409021369.06.zip# After (nightly from 2014-08-09): http://people.mozilla.org/~bgirard/cleopatra/?zippedProfile=http://people.mozilla.org/~wlachance/sps-profile-1409021721.19.zip# Let me know if you need anything else to help track this down.
Flags: needinfo?(matt.woodrow)
Assignee | ||
Comment 9•10 years ago
|
||
I'm not getting anywhere with reproducing this, so this patch just reverts the original changes for mobile only. The bug it was fixing doesn't happen there anyway (because Jeff fixed it differently) so this should be fine. Solving this using shaders is still the right long term fix :)
Attachment #8479510 -
Flags: review?(bas)
Flags: needinfo?(matt.woodrow)
Updated•10 years ago
|
Attachment #8479510 -
Flags: review?(bas) → review+
Assignee | ||
Comment 10•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/abcdfdab902b
Assignee | ||
Comment 11•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/90d25c4b3340
Comment 12•10 years ago
|
||
Matt, what is the status here? This is tracking 34.
Flags: needinfo?(matt.woodrow)
Comment 13•10 years ago
|
||
[Tracking Requested - why for this release]: tracking for fennec 34, but seems to have cross product dependencies
tracking-firefox34:
--- → ?
Reporter | ||
Comment 14•10 years ago
|
||
(In reply to Brad Lassey [:blassey] (use needinfo?) from comment #12) > Matt, what is the status here? This is tracking 34. Matt backed out the changes that caused the problem on Android. You can see things going back to normal around Aug 28: http://eideticker.mozilla.org/#/android/samsung-gn/mozilla-central/imgur/checkerboard/60
Flags: needinfo?(matt.woodrow)
Comment 15•10 years ago
|
||
This merge got dropped. https://hg.mozilla.org/mozilla-central/rev/abcdfdab902b https://hg.mozilla.org/mozilla-central/rev/90d25c4b3340
Comment 16•10 years ago
|
||
Confirmed that commits are in Aurora. Only remaining question is if there was a backout that didn't reference the bug number.
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox34:
--- → fixed
status-firefox35:
--- → fixed
OS: Linux → Android
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Updated•10 years ago
|
Updated•3 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
•