Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(152)

Unified Diff: src/core/SkRecordDraw.cpp

Issue 471063004: Stop sorting the results of SkBBH::search(). (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: rebase Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/core/SkPictureData.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkRecordDraw.cpp
diff --git a/src/core/SkRecordDraw.cpp b/src/core/SkRecordDraw.cpp
index 46241d72aca40cffe9239472b9c0b5cc72829057..17ac0d9cfde8eac381c285ae9a789659204ae32a 100644
--- a/src/core/SkRecordDraw.cpp
+++ b/src/core/SkRecordDraw.cpp
@@ -27,12 +27,6 @@ void SkRecordDraw(const SkRecord& record,
SkTDArray<void*> ops;
bbh->search(query, &ops);
- // FIXME: QuadTree doesn't send these back in the order we inserted them. :(
- // Also remove the sort in SkPictureData::getActiveOps()?
- if (ops.count() > 0) {
- SkTQSort(ops.begin(), ops.end() - 1, SkTCompareLT<void*>());
- }
-
SkRecords::Draw draw(canvas);
for (int i = 0; i < ops.count(); i++) {
if (NULL != callback && callback->abortDrawing()) {
« no previous file with comments | « src/core/SkPictureData.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698