commit ea9767356385866263a3164dcc1fb9e1633f57ef
author senorblanco <senorblanco@chromium.org> Thu Jun 09 12:43:30 2016 -0700
committer Commit bot <commit-bot@chromium.org> Thu Jun 09 12:43:30 2016 -0700
tree eb25103febdab3c49a11ab987ccb2c14af4a2ec0
parent 2843c42e594bf45f48a29f4195ec39aeeb92b2bc

SkPictureAnalyzer: expose the number of slow GPU commands.

BUG=614724
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2031243003

Review-Url: https://codereview.chromium.org/2031243003

include/core/SkPictureAnalyzer.h

diff --git a/include/core/SkPictureAnalyzer.h b/include/core/SkPictureAnalyzer.h
index b58807c..c46ca1e 100644
--- a/include/core/SkPictureAnalyzer.h
+++ b/include/core/SkPictureAnalyzer.h
@@ -48,6 +48,12 @@
      */
     bool suitableForGpuRasterization(const char** whyNot = nullptr) const;
 
+    /**
+     * Returns the number of commands which are slow to draw on the GPU, capped at the predicate
+     * max.
+     */
+    uint32_t numSlowGpuCommands() { return fNumSlowPaths; }
+
 private:
     uint32_t fNumSlowPaths;
 

src/core/SkPictureAnalyzer.cpp

diff --git a/src/core/SkPictureAnalyzer.cpp b/src/core/SkPictureAnalyzer.cpp
index 9f62d1d..b1d61ae 100644
--- a/src/core/SkPictureAnalyzer.cpp
+++ b/src/core/SkPictureAnalyzer.cpp
@@ -31,7 +31,7 @@
 }
 
 void SkPictureGpuAnalyzer::analyzePicture(const SkPicture* picture) {
-    if (!picture || veto_predicate(fNumSlowPaths)) {
+    if (!picture) {
         return;
     }
 
@@ -39,10 +39,6 @@
 }
 
 void SkPictureGpuAnalyzer::analyzeClipPath(const SkPath& path, SkRegion::Op op, bool doAntiAlias) {
-    if (veto_predicate(fNumSlowPaths)) {
-        return;
-    }
-
     const SkRecords::ClipPath clipOp = {
         SkIRect::MakeEmpty(), // Willie don't care.
         path,