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

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: 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..ed0fb07a60b325d4107a4bf0f9dbce6ecafa367c 100644
--- a/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
+++ b/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
@@ -51,6 +51,10 @@ public class WindowAndroid {
@Override
public void onVSync(VSyncMonitor monitor, long vsyncTimeMicros) {
if (mNativeWindowAndroid != 0) {
+ if (monitor.isVSyncSignalAvailable()) {
+ nativeUpdateVSyncPeriod(mNativeWindowAndroid,
Sami 2014/09/30 16:04:14 Instead of adding a new JNI call, please merge thi
jmanko 2014/10/01 09:36:51 Acknowledged.
+ mVSyncMonitor.getVSyncPeriodInMicroseconds());
+ }
nativeOnVSync(mNativeWindowAndroid, vsyncTimeMicros);
}
}
@@ -289,6 +293,7 @@ public class WindowAndroid {
private native long nativeInit(long vsyncPeriod);
private native void nativeOnVSync(long nativeWindowAndroid, long vsyncTimeMicros);
+ private native void nativeUpdateVSyncPeriod(long nativeWindowAndroid, long vsyncPeriod);
private native void nativeDestroy(long nativeWindowAndroid);
}

Powered by Google App Engine
This is Rietveld 408576698