commit bf3c602284f9a344faf185c3a5e94a264ba44c4f
author John Reck <jreck@google.com> Tue Jun 02 15:55:00 2015 -0700
committer John Reck <jreck@google.com> Tue Jun 02 16:13:56 2015 -0700
tree fdf404c8f4f4859290157229cc1689ed5b0b1df7
parent 233241f13d5d7f841d3db80408429d4d0a528c78

More colors, now with material colors

Bug: 18052916
Change-Id: I66c2573d796338ee7b7d0ddf240b90cc78a71ce5

libs/hwui/FrameInfoVisualizer.h

diff --git a/libs/hwui/FrameInfoVisualizer.h b/libs/hwui/FrameInfoVisualizer.h
index f62e34d..3fa4586 100644
--- a/libs/hwui/FrameInfoVisualizer.h
+++ b/libs/hwui/FrameInfoVisualizer.h
@@ -54,41 +54,21 @@
     void createData();
     void destroyData();
 
-    void addRect(Rect& r, float data, float* shapeOutput);
-    void prepareShapes(const int baseline);
+    void initializeRects(const int baseline);
+    void nextBarSegment(FrameInfoIndex start, FrameInfoIndex end);
     void drawGraph(OpenGLRenderer* canvas);
-    void drawCurrentFrame(OpenGLRenderer* canvas);
+    void drawCurrentFrame(const int baseline, OpenGLRenderer* canvas);
     void drawThreshold(OpenGLRenderer* canvas);
 
-    static inline float duration(nsecs_t start, nsecs_t end) {
-        float duration = ((end - start) * 0.000001f);
+    inline float duration(size_t index, FrameInfoIndex start, FrameInfoIndex end) {
+        nsecs_t ns_start = mFrameSource[index][start];
+        nsecs_t ns_end = mFrameSource[index][end];
+        float duration = ((ns_end - ns_start) * 0.000001f);
+        // Clamp to large to avoid spiking off the top of the screen
+        duration = duration > 50.0f ? 50.0f : duration;
         return duration > 0.0f ? duration : 0.0f;
     }
 
-    inline float recordDuration(size_t index) {
-        return duration(
-                mFrameSource[index][FrameInfoIndex::kIntendedVsync],
-                mFrameSource[index][FrameInfoIndex::kSyncStart]);
-    }
-
-    inline float prepareDuration(size_t index) {
-        return duration(
-                mFrameSource[index][FrameInfoIndex::kSyncStart],
-                mFrameSource[index][FrameInfoIndex::kIssueDrawCommandsStart]);
-    }
-
-    inline float issueDrawDuration(size_t index) {
-        return duration(
-                mFrameSource[index][FrameInfoIndex::kIssueDrawCommandsStart],
-                mFrameSource[index][FrameInfoIndex::kSwapBuffers]);
-    }
-
-    inline float swapBuffersDuration(size_t index) {
-        return duration(
-                mFrameSource[index][FrameInfoIndex::kSwapBuffers],
-                mFrameSource[index][FrameInfoIndex::kFrameCompleted]);
-    }
-
     ProfileType mType = ProfileType::None;
     float mDensity = 0;
 
@@ -105,7 +85,7 @@
      * OpenGLRenderer:drawRects() that makes up all the FrameTimingData:record
      * information.
      */
-    std::unique_ptr<float*> mRects;
+    std::unique_ptr<float[]> mRects;
 
     bool mShowDirtyRegions = false;
     SkRect mDirtyRegion;