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

Unified Diff: ui/base/android/window_android.cc

Issue 611313003: Use estimated vsync period on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove unused constant Created 6 years, 2 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 | « ui/base/android/window_android.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/base/android/window_android.cc
diff --git a/ui/base/android/window_android.cc b/ui/base/android/window_android.cc
index 490c6f3b545f2adfaabf67d3e4efb2766fed84f9..2fea4f1c00dc5f02dbc79d21f99727fae636778d 100644
--- a/ui/base/android/window_android.cc
+++ b/ui/base/android/window_android.cc
@@ -17,10 +17,8 @@ namespace ui {
using base::android::AttachCurrentThread;
using base::android::ScopedJavaLocalRef;
-WindowAndroid::WindowAndroid(JNIEnv* env, jobject obj, jlong vsync_period)
- : weak_java_window_(env, obj),
- compositor_(NULL),
- vsync_period_(base::TimeDelta::FromInternalValue(vsync_period)) {
+WindowAndroid::WindowAndroid(JNIEnv* env, jobject obj)
+ : weak_java_window_(env, obj), compositor_(NULL) {
}
void WindowAndroid::Destroy(JNIEnv* env, jobject obj) {
@@ -87,22 +85,27 @@ void WindowAndroid::Animate(base::TimeTicks begin_frame_time) {
WindowAndroidObserver, observer_list_, OnAnimate(begin_frame_time));
}
-void WindowAndroid::OnVSync(JNIEnv* env, jobject obj, jlong time_micros) {
+void WindowAndroid::OnVSync(JNIEnv* env,
+ jobject obj,
+ jlong time_micros,
+ jlong period_micros) {
base::TimeTicks frame_time(base::TimeTicks::FromInternalValue(time_micros));
+ base::TimeDelta vsync_period(
+ base::TimeDelta::FromMicroseconds(period_micros));
FOR_EACH_OBSERVER(
WindowAndroidObserver,
observer_list_,
- OnVSync(frame_time, vsync_period_));
+ OnVSync(frame_time, vsync_period));
if (compositor_)
- compositor_->OnVSync(frame_time, vsync_period_);
+ compositor_->OnVSync(frame_time, vsync_period);
}
// ----------------------------------------------------------------------------
// Native JNI methods
// ----------------------------------------------------------------------------
-jlong Init(JNIEnv* env, jobject obj, jlong vsync_period) {
- WindowAndroid* window = new WindowAndroid(env, obj, vsync_period);
+jlong Init(JNIEnv* env, jobject obj) {
+ WindowAndroid* window = new WindowAndroid(env, obj);
return reinterpret_cast<intptr_t>(window);
}
« no previous file with comments | « ui/base/android/window_android.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698