tree 6d362d91fd9bff8609d0141e0c621901f7adebc0
parent a5f9f1458eb0bfd776fc23677afd3d5d7e26a72b
author Kevin Lubick <kjlubick@google.com> 1595250064 -0400
committer Kevin Lubick <kjlubick@google.com> 1595252917 +0000

[gold] Add fork of infra-sk/sort-sk called sort-toggle-sk

sort-sk was not working for 2 reasons:
1) it was slow because it was always moving the nodes, which
lit-html can do faster when it keeps the nodes in place and just
changes their content (which it does when you do map).
2) It didn't work right when the data array changed (see
screencast in attached bug).

This behaves a little different in that you pass sort-toggle-sk
the array directly (instead of it sniffing it from the DOM)
and then it sorts that array (which the client uses to
render the nodes). Given the client control on whether to use
map or repeat() allows for the performance optimal solution
to be chosen (which speeds up sorting on the followup CL
by a factor of 2).

Change-Id: Ib04b2cacc22f538725c39650a67854a26c2293ea
Bug: skia:10504
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/303355
Reviewed-by: Kevin Lubick <kjlubick@google.com>