[Webkit-unassigned] [Bug 134208] Reduce synchronous layout triggered by JavaScript-based media controls

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=134208


Joseph Pecoraro <joepeck at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #233628|                            |review?, commit-queue?
               Flag|                            |




--- Comment #1 from Joseph Pecoraro <joepeck at webkit.org>  2014-06-23 12:47:59 PST ---
Created an attachment (id=233628)
 --> (https://bugs.webkit.org/attachment.cgi?id=233628&action=review)
[PATCH] Proposed Fix

Based on measurements from Andreas this improved PLT by ~0.5%. It reduces the number of calls to offsetWidth from 8 to 3 when just loading a page with a video and controls.

I see one possible issue with this patch, if the media element itself changes size the cached values may be wrong. That is because the timeline is flex based and we do not know when the element itself changed size. So that may be a regression that could be addressed later.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.