Android: Move scheduling logic to CompositorImpl

This adds two ways of scheduling:

- implicitly from LTHClient::ScheduleComposite():
This will post a task to composite immediately.

- explicitly (for Chrome) through SetNeedsComposite() where
there are no layer tree changes, but instead a Composite()
will be scheduled for vsync with the Layout() callback giving
the app a chance to update the tree.

TBR=skyostil@chromium.org
NOTRY=True

BUG=234173

Review URL: https://codereview.chromium.org/239963002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270231 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 31 deletions

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 8be2dd0..4a68dea 100644
--- a/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
+++ b/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java
@@ -32,7 +32,6 @@ public class WindowAndroid {
     // Native pointer to the c++ WindowAndroid object.
     private long mNativeWindowAndroid = 0;
     private final VSyncMonitor mVSyncMonitor;
-    private VSyncClient mVSyncClient = null;
 
     // A string used as a key to store intent errors in a bundle
     static final String WINDOW_CALLBACK_ERRORS = "window_callback_errors";
@@ -51,9 +50,6 @@ public class WindowAndroid {
     private final VSyncMonitor.Listener mVSyncListener = new VSyncMonitor.Listener() {
         @Override
         public void onVSync(VSyncMonitor monitor, long vsyncTimeMicros) {
-            if (mVSyncClient != null) {
-                mVSyncClient.onVSync(vsyncTimeMicros);
-            }
             if (mNativeWindowAndroid != 0) {
                 nativeOnVSync(mNativeWindowAndroid, vsyncTimeMicros);
             }
@@ -231,34 +227,9 @@ public class WindowAndroid {
         return false;
     }
 
-    /**
-     * An interface to receive VSync notifications from the window.
-     * The one and only client is set with setVSyncClient(client).
-     */
-    public interface VSyncClient {
-        /**
-         * Called very soon after the start of the display's vertical sync period.
-         * @param vsyncTimeMicros Absolute frame time in microseconds.
-         */
-        void onVSync(long vsyncTimeMicros);
-    }
-
-    /**
-     * Sets the VSyncClient.
-     * @param client The client receiving VSync notifications.
-     */
-    public void setVSyncClient(VSyncClient client) {
-        assert mVSyncClient == null || client == null;
-        mVSyncClient = client;
-    }
-
-    /**
-     * Request a VSync callback.
-     * VSyncClient.onVSync() will be called at least once.
-     */
     @CalledByNative
-    public void requestVSyncUpdate() {
-        mVSyncMonitor.requestUpdate();
+    private void requestVSyncUpdate() {
+       mVSyncMonitor.requestUpdate();
     }
 
     /**