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

Unified Diff: ui/android/java/src/org/chromium/ui/base/WindowAndroid.java

Issue 611313003: Use estimated vsync period on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review follow-up Created 6 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
Index: ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
diff --git a/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java b/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
index 6d3f3a6d1538a90ed7137b6f992c74610cfc56d8..ef56966f652ec350c8d4baf08af43bea7006acad 100644
--- a/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
+++ b/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
@@ -51,7 +51,9 @@ public class WindowAndroid {
@Override
public void onVSync(VSyncMonitor monitor, long vsyncTimeMicros) {
if (mNativeWindowAndroid != 0) {
- nativeOnVSync(mNativeWindowAndroid, vsyncTimeMicros);
+ nativeOnVSync(mNativeWindowAndroid,
+ vsyncTimeMicros,
+ mVSyncMonitor.getVSyncPeriodInMicroseconds());
}
}
};
@@ -288,7 +290,9 @@ public class WindowAndroid {
}
private native long nativeInit(long vsyncPeriod);
- private native void nativeOnVSync(long nativeWindowAndroid, long vsyncTimeMicros);
+ private native void nativeOnVSync(long nativeWindowAndroid,
+ long vsyncTimeMicros,
+ long vsyncPeriod);
Sami 2014/10/01 11:42:49 nit: vsyncPeriodMicros.
private native void nativeDestroy(long nativeWindowAndroid);
}

Powered by Google App Engine
This is Rietveld 408576698