diff options
author | joshualitt <joshualitt@chromium.org> | 2016-03-08 10:43:41 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-08 10:43:42 -0800 |
commit | b0666ad3a9e99ce1a6e6d9ea69ff9f1ddbe74594 (patch) | |
tree | 001b9a724edb773a657ced2446fdb9db996fcc17 /include/private/GrAuditTrail.h | |
parent | d8a4f772115bbb9d50c0e73613bdd879833da690 (diff) |
Fix up GrAuditTrail to allow arbitrary reordering
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1765123002
Review URL: https://codereview.chromium.org/1765123002
Diffstat (limited to 'include/private/GrAuditTrail.h')
-rw-r--r-- | include/private/GrAuditTrail.h | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/include/private/GrAuditTrail.h b/include/private/GrAuditTrail.h index 4d2d99aa9a..3bb7bea435 100644 --- a/include/private/GrAuditTrail.h +++ b/include/private/GrAuditTrail.h @@ -84,11 +84,9 @@ public: fCurrentStackTrace.push_back(SkString(framename)); } - void addBatch(const char* name, const SkRect& bounds); + void addBatch(const GrBatch* batch); - void batchingResultCombined(GrBatch* combiner); - - void batchingResultNew(GrBatch* batch); + void batchingResultCombined(const GrBatch* consumer, const GrBatch* consumed); // Because batching is heavily dependent on sequence of draw calls, these calls will only // produce valid information for the given draw sequence which preceeded them. @@ -153,10 +151,9 @@ private: template <typename T> static void JsonifyTArray(SkString* json, const char* name, const T& array, bool addComma); - - Batch* fCurrentBatch; + BatchPool fBatchPool; - SkTHashMap<GrBatch*, int> fIDLookup; + SkTHashMap<uint32_t, int> fIDLookup; SkTHashMap<int, Batches*> fClientIDLookup; BatchList fBatchList; SkTArray<SkString> fCurrentStackTrace; @@ -177,13 +174,13 @@ private: #define GR_AUDIT_TRAIL_RESET(audit_trail) \ //GR_AUDIT_TRAIL_INVOKE_GUARD(audit_trail, fullReset); -#define GR_AUDIT_TRAIL_ADDBATCH(audit_trail, batchname, bounds) \ - GR_AUDIT_TRAIL_INVOKE_GUARD(audit_trail, addBatch, batchname, bounds); +#define GR_AUDIT_TRAIL_ADDBATCH(audit_trail, batch) \ + GR_AUDIT_TRAIL_INVOKE_GUARD(audit_trail, addBatch, batch); -#define GR_AUDIT_TRAIL_BATCHING_RESULT_COMBINED(audit_trail, combiner) \ - GR_AUDIT_TRAIL_INVOKE_GUARD(audit_trail, batchingResultCombined, combiner); +#define GR_AUDIT_TRAIL_BATCHING_RESULT_COMBINED(audit_trail, combineWith, batch) \ + GR_AUDIT_TRAIL_INVOKE_GUARD(audit_trail, batchingResultCombined, combineWith, batch); #define GR_AUDIT_TRAIL_BATCHING_RESULT_NEW(audit_trail, batch) \ - GR_AUDIT_TRAIL_INVOKE_GUARD(audit_trail, batchingResultNew, batch); + // Doesn't do anything now, one day... #endif |