[Webkit-unassigned] [Bug 135313] Web Inspector: Timelines performance is very slow, has many forced layouts

[bugzilla-daemon at webkit.org]

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


Joseph Pecoraro <joepeck at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #235555|0                           |1
        is obsolete|                            |
 Attachment #235555|review?                     |
               Flag|                            |
 Attachment #235560|                            |review?, commit-queue?
               Flag|                            |




--- Comment #5 from Joseph Pecoraro <joepeck at webkit.org>  2014-07-25 20:08:10 PST ---
Created an attachment (id=235560)
 --> (https://bugs.webkit.org/attachment.cgi?id=235560&action=review)
[PATCH] Proposed Fix

Better. It turns out showEmptyPlaceholder could also benefit from the early return if nothing changed. This is another huge win if the time range selection doesn't start at 0.0. Otherwise we would have hit the same forced layout loop calling this over and over. Note, the messages is always the same so maybe we can simplify this a bit more.

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