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

Unified Diff: tools/gpu/gpu_tools/scroll.js

Issue 10916227: More polish in prep for CrOS support (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: for landing Created 8 years, 3 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 | « tools/gpu/gpu_tools/multi_page_benchmark_unittest.py ('k') | tools/gpu/gpu_tools/scrolling_benchmark.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gpu/gpu_tools/scroll.js
diff --git a/tools/gpu/gpu_tools/scroll.js b/tools/gpu/gpu_tools/scroll.js
index c2a8815d50e2b6937119610a180b2fd2349ced6f..a8b35e8d99938775515a4f0b11cae0fe2c55b7e1 100644
--- a/tools/gpu/gpu_tools/scroll.js
+++ b/tools/gpu/gpu_tools/scroll.js
@@ -122,7 +122,7 @@
RafRenderingStats.prototype.get = function() {
var results = {};
results.numAnimationFrames = this.frameTimes_.length - 1;
- results.numFramesSentToScreen = results.numFramesSentToScreen;
+ results.numFramesSentToScreen = results.numAnimationFrames;
results.droppedFrameCount = this.getDroppedFrameCount_(this.frameTimes_);
results.totalTimeInSeconds = (
this.frameTimes_[this.frameTimes_.length - 1] -
@@ -189,7 +189,8 @@
ScrollTest.prototype.startPass_ = function() {
this.element_.scrollTop = 0;
- if (window.chrome && chrome.gpuBenchmarking)
+ if (window.chrome && chrome.gpuBenchmarking &&
+ chrome.gpuBenchmarking.renderingStats)
this.renderingStats_ = new GpuBenchmarkingRenderingStats();
else
this.renderingStats_ = new RafRenderingStats();
« no previous file with comments | « tools/gpu/gpu_tools/multi_page_benchmark_unittest.py ('k') | tools/gpu/gpu_tools/scrolling_benchmark.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698