[Webkit-unassigned] [Bug 33579] Web Inspector: speed-up Timeline panel scrolling

[bugzilla-daemon at webkit.org]

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


Pavel Feldman <pfeldman at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #46436|                            |review-
               Flag|                            |




--- Comment #2 from Pavel Feldman <pfeldman at chromium.org>  2010-01-13 02:08:21 PST ---
(From update of attachment 46436)
Looks good. Few suggestions on the naming before it lands:

> +    this._updateBoundaries = false;

this._updateBoundaries -> this._boundariesAreValid (and inverse the meaning)

>  
>      _scheduleRefresh: function(immediate)
>      {

immediate -> preserveBoundaries

> +    _refreshRecords: function(updateBoundaries)

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