1050725 - Reduce overpaint of the email ap

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: email.patch

James, this shape some of the overpaint of the app and should result into a few fps win on the main list as well as less memory allocated for it by default.
This should also not break any transitions.

Comment 1

[Andrew Sutherland [:asuth] (he/him)]

(In reply to Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please) from comment #0)
> This should also not break any transitions.

Can you clarify if this determination is made by code inspection or if it was on-device testing, and if so, which devices?  (Also whether frame trees were consulted, etc.)

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Andrew Sutherland [:asuth] from comment #1)
> (In reply to Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails,
> needinfo? please) from comment #0)
> > This should also not break any transitions.
> 
> Can you clarify if this determination is made by code inspection or if it
> was on-device testing, and if so, which devices?  (Also whether frame trees
> were consulted, etc.)

It was testing on the flame. Let me know if you see any broken animations.

Comment 3

[James Burke [:jrburke]]

Comment on attachment 8469953 [details] [diff] [review]
email.patch

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

I tried on Flame, using pvt build and latest gaia master as of the writing of this note. Showed FPS and overpaint on device. Overpaint in general seemed to go down by 100-ish, and a few more fps were gained.

Main test was going from message_list to message_reader and back. Numbers are from going back to message list from the message reader, as it was a fairly reliable thing to capture: both card contents were static at that point.

Before: 
fps: 15-18
overpaint: 488


After:
fps:22
overpain: 393

Similar rough diffs for other cards.

I also converted the translateY(0) to none and saw similar gains, the overpaint changes were the most noticeable difference.

:vingtetun, if it helps, I can formalize this as a pull request and merge. If I do not hear anything, that is what I will do on Wednesday Aug 13.

::: apps/email/style/common.css
@@ -60,4 @@
>  }
>  
>  .card.center.anim-vertical {
>    transform: translateY(0);

I also switched this to transform: none in my local test, seemed to have similar benefits for the menu/folder drawer.

Comment 4

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to James Burke [:jrburke] from comment #3)
> :vingtetun, if it helps, I can formalize this as a pull request and merge.
> If I do not hear anything, that is what I will do on Wednesday Aug 13.


Please do. I'm on PTO :)

Comment 5

[James Burke [:jrburke]]

Sorry for the delay, I was stuck on a refactor branch.

Merged in master:
https://github.com/mozilla-b2g/gaia/commit/f695792496326679702a0b2f48a9c384fd47473a

from pull request:
https://github.com/mozilla-b2g/gaia/pull/22899

Comment 6

[James Burke [:jrburke]]

As documented in bug 1063311, this change caused some weird layer changes for an activity+cancel flow. Not entirely sure about the root cause, will keep this bug open to track it down, then reapply this change once the root issue is discovered.

Comment 7

[BMO Automation]

Firefox OS is not being worked on