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

Unified Diff: cc/test/begin_frame_args_test.cc

Issue 1133673004: cc: Heuristic for Renderer latency recovery (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add SchedulerStateMachine::SwapThrottled Created 5 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
Index: cc/test/begin_frame_args_test.cc
diff --git a/cc/test/begin_frame_args_test.cc b/cc/test/begin_frame_args_test.cc
index 7b2974867469d3803d029dedc483dff233290edb..a17fb4b3c5ce76e70ee0311906983f3ad11cc8ca 100644
--- a/cc/test/begin_frame_args_test.cc
+++ b/cc/test/begin_frame_args_test.cc
@@ -20,7 +20,8 @@ BeginFrameArgs CreateBeginFrameArgsForTesting(
base::TimeTicks frame_time) {
return BeginFrameArgs::Create(
location, frame_time,
- frame_time + (BeginFrameArgs::DefaultInterval() / 2),
+ frame_time + BeginFrameArgs::DefaultInterval() -
+ BeginFrameArgs::DefaultEstimatedParentDrawTime(),
mithro-old 2015/05/27 04:40:15 nit: I wonder if this should be in a separate CL?
brianderson 2015/05/27 23:22:32 Will split it out.
BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL);
}
@@ -60,7 +61,8 @@ BeginFrameArgs CreateBeginFrameArgsForTesting(
scoped_refptr<TestNowSource> now_src) {
base::TimeTicks now = now_src->Now();
return BeginFrameArgs::Create(
- location, now, now + (BeginFrameArgs::DefaultInterval() / 2),
+ location, now, now + BeginFrameArgs::DefaultInterval() -
+ BeginFrameArgs::DefaultEstimatedParentDrawTime(),
mithro-old 2015/05/27 04:40:15 nit: Same as above....
BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL);
}

Powered by Google App Engine
This is Rietveld 408576698