RESOLVED FIXED103611
[Shadow] We should have a performance test of distribution for changing select attribute. 
https://bugs.webkit.org/show_bug.cgi?id=103611
Summary [Shadow] We should have a performance test of distribution for changing selec...
Shinya Kawanaka
Reported 2012-11-29 00:51:19 PST
We would like to compare performance between (a) distributing elements using java script and (b) distributing elements using ShadowDOM.
Attachments
Patch (4.72 KB, patch)
2012-11-29 01:15 PST, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Patch (4.90 KB, patch)
2012-11-29 02:50 PST, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Patch (5.31 KB, patch)
2012-11-29 02:55 PST, Shinya Kawanaka 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Shinya Kawanaka
Comment 1 2012-11-29 00:53:29 PST
This is a case when select attribute is changed.
Shinya Kawanaka
Comment 2 2012-11-29 01:15:13 PST
Created attachment 176671 [details] Patch
Hajime Morrita
Comment 3 2012-11-29 01:38:14 PST
Comment on attachment 176671 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=176671&action=review Could you add these to PerfomanceTests/Skipped so that they don't run regularly? I think this isn't ready for prime time yet. > PerformanceTests/ShadowDOM/ChangingSelect.html:61 > + description: "Measure Time without", What is this meant to say? > PerformanceTests/ShadowDOM/ChangingSelectWithoutShadow.html:64 > + description: "Measure Time without", What is this meant to say?
WebKit Review Bot
Comment 4 2012-11-29 02:16:11 PST
Comment on attachment 176671 [details] Patch Attachment 176671 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/15014857 New failing tests: inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html
Shinya Kawanaka
Comment 5 2012-11-29 02:39:11 PST
(In reply to comment #3) > (From update of attachment 176671 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=176671&action=review > > Could you add these to PerfomanceTests/Skipped so that they don't run regularly? I think this isn't ready for prime time yet. > > > PerformanceTests/ShadowDOM/ChangingSelect.html:61 > > + description: "Measure Time without", > > What is this meant to say? > > > PerformanceTests/ShadowDOM/ChangingSelectWithoutShadow.html:64 > > + description: "Measure Time without", > > What is this meant to say? Oops...
Shinya Kawanaka
Comment 6 2012-11-29 02:50:52 PST
Created attachment 176685 [details] Patch
Shinya Kawanaka
Comment 7 2012-11-29 02:55:03 PST
Created attachment 176686 [details] Patch
WebKit Review Bot
Comment 8 2012-11-29 03:27:22 PST
Comment on attachment 176686 [details] Patch Clearing flags on attachment: 176686 Committed r136120: <http://trac.webkit.org/changeset/136120>
WebKit Review Bot
Comment 9 2012-11-29 03:27:25 PST
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.