RESOLVED INVALID 103753
Web Inspector: cache scrollTop and clientHeight in the text editor. 
https://bugs.webkit.org/show_bug.cgi?id=103753
Summary Web Inspector: cache scrollTop and clientHeight in the text editor.
Pavel Feldman
Reported 2012-11-30 09:20:59 PST
Caching them to reduce the amount of forced layouts.
Attachments
Patch (13.40 KB, patch)
2012-11-30 13:55 PST, Pavel Feldman 		no flags
DetailsFormatted DiffDiff
Patch (15.02 KB, patch)
2012-11-30 14:58 PST, Pavel Feldman 		vsevik: review-
gtk-ews: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Pavel Feldman
Comment 1 2012-11-30 13:55:21 PST
Created attachment 177021 [details] Patch
Pavel Feldman
Comment 2 2012-11-30 14:58:53 PST
Created attachment 177031 [details] Patch
kov's GTK+ EWS bot
Comment 3 2012-11-30 15:06:25 PST
Comment on attachment 177031 [details] Patch Attachment 177031 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/15083125
Build Bot
Comment 4 2012-11-30 21:13:56 PST
Comment on attachment 177031 [details] Patch Attachment 177031 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/15067430 New failing tests: inspector/editor/text-editor-selection.html
WebKit Review Bot
Comment 5 2012-12-02 05:27:40 PST
Comment on attachment 177031 [details] Patch Attachment 177031 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/15101152 New failing tests: inspector/editor/text-editor-selection.html
Vsevolod Vlasov
Comment 6 2012-12-10 01:34:37 PST
Comment on attachment 177031 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=177031&action=review > Source/WebCore/inspector/front-end/DefaultTextEditor.js:-932 > - delete this._repaintAllTimer; I don't think this should have been removed here.
Note You need to log in before you can comment on or make changes to this bug.