RESOLVED FIXED 60959
Web Inspector: panel switching is too slow 
https://bugs.webkit.org/show_bug.cgi?id=60959
Summary Web Inspector: panel switching is too slow
Ilya Tikhonovsky
Reported 2011-05-17 07:58:48 PDT
%subj%
Attachments
[patch] initial version (1.19 KB, patch)
2011-05-17 08:02 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
[patch] second version (3.50 KB, patch)
2011-07-12 10:33 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
[patch] third version (3.63 KB, patch)
2011-07-13 23:29 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2011-05-17 08:02:21 PDT
Created attachment 93765 [details] [patch] initial version
Simon Fraser (smfr)
Comment 2 2011-05-18 12:55:40 PDT
Comment on attachment 93765 [details] [patch] initial version Did you try visibility:hidden rather than opacity? It might be faster
Ilya Tikhonovsky
Comment 3 2011-05-18 13:19:22 PDT
(In reply to comment #2) > (From update of attachment 93765 [details]) > Did you try visibility:hidden rather than opacity? It might be faster I've checked this right now and get the next values My test case: Inspector opened for gmail page with expanded body children items I'm switching inspector from resource panel to elements panel and back display:none r2e ~550ms, e2r ~8.5sec visibility: hidden: r2e ~340ms, e2r ~125ms opacity: 0: r2e ~150ms, e2r ~37ms
Yury Semikhatsky
Comment 4 2011-05-26 07:17:24 PDT
Comment on attachment 93765 [details] [patch] initial version As discussed offline clearing r? while Ilya is looking for a better solution.
Ilya Tikhonovsky
Comment 5 2011-07-12 10:33:25 PDT
Created attachment 100507 [details] [patch] second version Looks like we have no better solution for panel switching.
Ilya Tikhonovsky
Comment 6 2011-07-13 23:29:45 PDT
Created attachment 100772 [details] [patch] third version there was a problem with mouse events. It happened with scripts panel when it is opened after profile panel.
Pavel Feldman
Comment 7 2011-07-13 23:30:56 PDT
Comment on attachment 100772 [details] [patch] third version Can we "inherit" all views from ".view" and specify .visible once? Panel inherits from View as well, so it'll work for it.
Ilya Tikhonovsky
Comment 8 2011-07-14 05:34:27 PDT
Comment on attachment 93765 [details] [patch] initial version After offline discussion it was decided to use opacity only for panels. It is not possible to use such solution for all the views because in that case we need to manage the visible state of each the view explicitly.
WebKit Review Bot
Comment 9 2011-07-14 12:51:54 PDT
Comment on attachment 93765 [details] [patch] initial version Clearing flags on attachment: 93765 Committed r91024: <http://trac.webkit.org/changeset/91024>
WebKit Review Bot
Comment 10 2011-07-14 12:52:00 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.