Closed Bug 599576 Opened 9 years ago Closed 9 years ago

Talos Regression :( Ts increase 22.1% on Nokia n900 mobile

Categories

(Firefox for Android Graveyard :: General, defect)

All
Maemo
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file)

Previous: avg 7752.540 stddev 74.267 of 30 runs up to revision 54af7f2df238
New     : avg 9468.978 stddev 14.813 of 5 runs since revision 6ce6d5e09a55
Change  : +1716.438 (22.1% / z=23.112)
Graph   : http://mzl.la/9MspZZ

Changeset range: http://hg.mozilla.org/mobile-browser/pushloghtml?fromchange=54af7f2df238&tochange=6ce6d5e09a55

Bugs:
  * bug 599512 - Partial black thumbnails on viewports with width != 800
  * bug 599053 - text typed in awesome bar shows up on previously focused form field
  * bug 599501 - Browser fights me when I try to zoom
Attached patch patchSplinter Review
This patch seems to recover much of the lost Ts and still seems to not regress bug 599512. I could only try this locally on my N900. We'll need to push it to see if tinderbox likes it as much as my N900 did.
Assignee: nobody → mark.finkle
Attachment #478502 - Flags: review?(webapps)
Comment on attachment 478502 [details] [diff] [review]
patch

This looks good, and does not regress the black thumbnails.

I'd also be curious whether thumbnailing a smaller area is faster.  For example, we could set "width = Math.max(800, width)" in documenttab.updateThumbnail.
Attachment #478502 - Flags: review?(webapps) → review+
tracking-fennec: --- → ?
OS: Linux → Maemo
Hardware: x86_64 → All
This patch was not enough to fix the Ts regression. I backed out all suspected patches (see comment 0). I'll land them one at a time to see which one caused the regression.
Ts regression is gone and we plan on fixing bug 599053 using a different strategy. Closing this bug as fixed by the backout.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.