Summary: | Make some small improvements involving sorting | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Darin Adler <darin> | ||||
Component: | WebCore Misc. | Assignee: | Darin Adler <darin> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andersca, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, macpherson, menard, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=214883 | ||||||
Attachments: |
|
Description
Darin Adler
2020-08-18 11:47:26 PDT
Created attachment 406797 [details]
Patch
Comment on attachment 406797 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=406797&action=review > Source/WebCore/page/PerformanceObserver.cpp:94 > + if (oldEnd != end) { Just realized I don’t think this if statement is needed. I think both algorithms below are super-efficient when the ranges are empty. Committed r266027: <https://trac.webkit.org/changeset/266027> |