Description
Ilya Tikhonovsky
2010-01-13 00:29:52 PST
Created attachment 46436 [details]
Patch file with changes in TimelinePanel.js and WebCore/ChangeLog
Comment on attachment 46436 [details] Patch file with changes in TimelinePanel.js and WebCore/ChangeLog 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) Created attachment 46454 [details]
Patch for TimelinePanel.js. Style was adjusted a bit. Problem with overview pane was fixed.
Comment on attachment 46454 [details] Patch for TimelinePanel.js. Style was adjusted a bit. Problem with overview pane was fixed. > + this._boundariesAreValid &= preserveBoundaries You did not initialize this._boundariesAreValid field. Otherwise r+ Created attachment 46458 [details]
Patch has changes for TimelinePanel.js. The initialization of _boundariesAreValid was added.
Created attachment 46460 [details]
Patch has changes for TimelinePanel.js. The initialization of _boundariesAreValid was added. ChangeLog was fixed.
Attachment 46460 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/ChangeLog:5: Line contains tab character. [whitespace/tab] [5]
Total errors found: 1
Created attachment 46461 [details]
Patch has changes for TimelinePanel.js. The initialization of _boundariesAreValid was added. ChangeLog was fixed.
Created attachment 46462 [details]
Patch has changes for TimelinePanel.js. The initialization of _boundariesAreValid was added. ChangeLog was fixed.
Comment on attachment 46462 [details] Patch has changes for TimelinePanel.js. The initialization of _boundariesAreValid was added. ChangeLog was fixed. Clearing flags on attachment: 46462 Committed r53188: <http://trac.webkit.org/changeset/53188> All reviewed patches have been landed. Closing bug. |