it is bad from the performance point of view.
Created attachment 105620 [details] [patch] initial version
Comment on attachment 105620 [details] [patch] initial version View in context: https://bugs.webkit.org/attachment.cgi?id=105620&action=review > Source/WebCore/inspector/front-end/ConsoleView.js:286 > + if (!this._isScrollIntoViewScheduled() && ((msg instanceof WebInspector.ConsoleCommandResult) || this.messagesElement.isScrolledToBottom())) this._scrollIntoViewTimer is already checked in _scheduleScrollIntoView(), isn't it?
Comment on attachment 105620 [details] [patch] initial version View in context: https://bugs.webkit.org/attachment.cgi?id=105620&action=review >> Source/WebCore/inspector/front-end/ConsoleView.js:286 >> + if (!this._isScrollIntoViewScheduled() && ((msg instanceof WebInspector.ConsoleCommandResult) || this.messagesElement.isScrolledToBottom())) > > this._scrollIntoViewTimer is already checked in _scheduleScrollIntoView(), isn't it? the problem is in this.messagesElement.isScrolledToBottom call. This call is forcing style recalculation. In the old code we do style recalculation even if we already scheduled the scroll action.
Comment on attachment 105620 [details] [patch] initial version View in context: https://bugs.webkit.org/attachment.cgi?id=105620&action=review >>> Source/WebCore/inspector/front-end/ConsoleView.js:286 >>> + if (!this._isScrollIntoViewScheduled() && ((msg instanceof WebInspector.ConsoleCommandResult) || this.messagesElement.isScrolledToBottom())) >> >> this._scrollIntoViewTimer is already checked in _scheduleScrollIntoView(), isn't it? > > the problem is in this.messagesElement.isScrolledToBottom call. This call is forcing style recalculation. > In the old code we do style recalculation even if we already scheduled the scroll action. isScrolledToBottom() should be called *before* the new message has been added.
Created attachment 105632 [details] [patch] second iteration
Comment on attachment 105632 [details] [patch] second iteration Clearing flags on attachment: 105632 Committed r94178: <http://trac.webkit.org/changeset/94178>
All reviewed patches have been landed. Closing bug.