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

Unified Diff: chrome/test/perf/rendering/latency_tests.cc

Issue 15058004: cc: Rename VSync to BeginFrame (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebase Created 7 years, 7 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 | « cc/trees/thread_proxy.cc ('k') | content/browser/android/content_view_core_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/perf/rendering/latency_tests.cc
diff --git a/chrome/test/perf/rendering/latency_tests.cc b/chrome/test/perf/rendering/latency_tests.cc
index fdd00d77ecae94a7e0c32b10fede9f143a980704..af3c761c0b19b8daebe5809f97b4dbce5b3e28fc 100644
--- a/chrome/test/perf/rendering/latency_tests.cc
+++ b/chrome/test/perf/rendering/latency_tests.cc
@@ -99,9 +99,9 @@ const int kInputsPerFrame = 16;
const int kClearColorGreen = 137;
const int kMouseY = 5;
-// Don't analyze begin frames that may be inaccurate. Latencies can be as high
+// Don't analyze start frames that may be inaccurate. Latencies can be as high
// as 5 frames or so, so skip the first 6 frames to get more accurate results.
-const int kIgnoreBeginFrames = 6;
+const int kIgnoreStartFrames = 6;
// Don't analyze end frames that may be inaccurate.
const int kIgnoreEndFrames = 4;
// Minimum frames to produce an answer.
@@ -478,7 +478,7 @@ double LatencyTest::CalculateLatency() {
++swap_count;
// Don't analyze first few swaps, because they are filling the rendering
// pipeline and may be unstable.
- if (swap_count > kIgnoreBeginFrames) {
+ if (swap_count > kIgnoreStartFrames) {
// First, find the beginning of this swap.
size_t begin_swap_pos = 0;
EXPECT_TRUE(FindLastOf(events, query_begin_swaps_, end_swap_pos,
« no previous file with comments | « cc/trees/thread_proxy.cc ('k') | content/browser/android/content_view_core_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698