824088 - With old (pre-821221) versions of gaia SMS app, we sometimes don't paint text on "Deselect all" "Select all" buttons

Status: RESOLVED WONTFIX

(Core :: Graphics, defect)

Description

[Daniel Holbert [:dholbert]]

I'm splitting this bug off for the platform issue identified in bug 821221 comment 12:
> So -- I did the following:
> 
> (a) Toggled gfxUtils::sDumpPaintList to dump display lists
> (b) Reproduced the bug (on the "Select all" button), & saved the last
> display-list
> (c) Tapped the "Select all" button (w/ missing text) and then "Deselect
> all", which returned me to the same selection-state as in (b), except with
> the button-text now visible, and saved that last display list.
> 
> From comparing the display lists, mattwoodrow & I discovered that the
> display lists at the end of (b) and (c) were **the same**.  (There was only
> one difference, and it wasn't significant -- we replaced the old
> opacity-layer around the (disabled) "Deselect all" button with a new one,
> because it temporarily lost its opacity when it was enabled in the middle of
> step (c).)
> 
> mattwoodrow says this indicates that this is a painting bug.  (since, given
> the same display list, we're painting different things)  I don't really know
> how to debug that -- I suspect it's tricky, and mattwoodrow suggests that it
> might be best to throw a cairo expert at it.  (jrmuizel?)  I don't know how
> soon that can happen or how long that will take.

We landed a gaia workaround on that bug, and I'm splitting this bug off for the actual platform issue.

Hopefully we can come up with a reduced testcase at some point; until that happens, the SMS app without the workaround (before https://github.com/mozilla-b2g/gaia/pull/7117 was merged) should suffice as a testcase, with the STR from bug 821221 comment 0.

Comment 1

[Walter Chen[:ypwalter][:wachen]]

It's not older gaia. I think I always use the most up-to-date gaia for testing.

Comment 2

[Daniel Holbert [:dholbert]]

I don't know what Comment 1 means -- I think you might be misunderstanding. (I didn't say anything about "older gaia" -- what I said was that gaia's SMS app should reproduce this bug, up to the point (today) where that pull request was merged).

Comment 3

[Daniel Holbert [:dholbert]]

(ah, now I see the summary says "old" -- sorry if that was confusing. Adding a clarification to say that "old" means "before bug 821221's workaround was merged."  :) (So as of today, that's not *very* old, but the point is it won't affect current versions of the SMS app anymore.))

Comment 4

[Sylvestre Ledru [:Sylvestre]]

Closing as we are not working on Firefox OS anymore.