If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

AppWindow destroy triggers flush/restyles

NEW
Unassigned

Status

Firefox OS
Gaia::System::Window Mgmt
3 years ago
3 years ago

People

(Reporter: gerard, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Created attachment 8497349 [details]
profile_captured_highcpu_1.sym

Observed by profiling B2G process on a Nexus S, on a build from sept. 25th.

Profile: http://people.mozilla.org/~bgirard/cleopatra/#report=9f32e34cd7c2628d1f7e26a9bf8bc3a0c6fa06a5

Restyles is triggered by aw_destroy()
(Reporter)

Comment 1

3 years ago
Etienne, maybe you can help me to have a look?
Flags: needinfo?(etienne)
When we destroy an app (oom-kill or user-kill) we remove the dead iframe from the DOM so we're going to reflow anyway.

Was the flush particularly long? Not sure there's a bug here.
Flags: needinfo?(etienne)
(Reporter)

Comment 3

3 years ago
Well I noticed the Restyles call, but it's your call to consider whether there's a bug or not according to the profiling I provided :)
You need to log in before you can comment on or make changes to this bug.