diff options
author | egraether@chromium.org <egraether@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-05 21:48:15 +0000 |
---|---|---|
committer | egraether@chromium.org <egraether@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-05 21:48:15 +0000 |
commit | af97816e82e643e37c69656f55b9cfd12ba72eec (patch) | |
tree | f5157391530df0862bde3481fbc1775179784da8 /cc/frame_rate_counter.h | |
parent | 89e711aa9c8dc7e38e7e1712c9598cf9391ad800 (diff) | |
download | chromium_src-af97816e82e643e37c69656f55b9cfd12ba72eec.zip chromium_src-af97816e82e643e37c69656f55b9cfd12ba72eec.tar.gz chromium_src-af97816e82e643e37c69656f55b9cfd12ba72eec.tar.bz2 |
this change improves the following points of the FPS counter:
-cleaner design: less colors, text arranged above the graph, deviation number right aligned
-added transparency: FPS counter does not completely cover the webpage
-less frequent number updates: makes the numbers easier to read
-indicator line at 60fps
comparison images: https://docs.google.com/folder/d/0B8Y78t3tjy1XZk1xdWx6VjN5aFE/edit
Please download the patch and provide a screenshot from your system to test the layout.
Review URL: https://chromiumcodereview.appspot.com/11272042
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166041 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/frame_rate_counter.h')
-rw-r--r-- | cc/frame_rate_counter.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/cc/frame_rate_counter.h b/cc/frame_rate_counter.h index f9b770a..80697da 100644 --- a/cc/frame_rate_counter.h +++ b/cc/frame_rate_counter.h @@ -25,7 +25,8 @@ public: // n = 0 returns the oldest frame retained in the history, // while n = timeStampHistorySize() - 1 returns the timestamp most recent frame. - base::TimeTicks timeStampOfRecentFrame(int n); + // FIXME: Returns most recent timestamp for n = 0 when called between markBeginningOfFrame and markEndOfFrame calls. + base::TimeTicks timeStampOfRecentFrame(int n) const; // This is a heuristic that can be used to ignore frames in a reasonable way. Returns // true if the given frame interval is too fast or too slow, based on constant thresholds. @@ -51,7 +52,7 @@ private: // FIXME: Determine this threshold based on monitor refresh rate, crbug.com/138642. static const double kDroppedFrameTime; - static const int kTimeStampHistorySize = 120; + static const int kTimeStampHistorySize = 170; int m_currentFrameNumber; base::TimeTicks m_timeStampHistory[kTimeStampHistorySize]; |