Moving logic for System UI state changes into DesktopCanvas class

remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java

Index: remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java
diff --git a/remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java b/remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java
index 2cc913d79301d30d85f9fbd142c61015f4d8371d..ec6a46774ecfa0d1486b4438fa053fd6f98c9e7d 100644
--- a/remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java
+++ b/remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java
@@ -125,40 +125,37 @@ public class DesktopCanvas {
     }
 
     /**
-     * Sets the offset values used to calculate the space used by System UI.
+     * Handles System UI size and visibility changes.
      *
-     * @param left The space used by System UI on the left edge of the screen.
-     * @param top The space used by System UI on the top edge of the screen.
-     * @param right The space used by System UI on the right edge of the screen.
-     * @param bottom The space used by System UI on the bottom edge of the screen.
+     * @param parameter The set of values defining the current System UI state.
      */
-    public void setSystemUiOffsetValues(int left, int top, int right, int bottom) {
-        mSystemUiScreenSize.set(left, top, right, bottom);
-
-        if (mAdjustViewportForSystemUi) {
-            // Adjust the cursor position to ensure it's visible when large System UI (defined as
-            // 1/3 or more of the total screen size) is displayed.  This is typically the Soft
-            // Keyboard. Without this change, it is difficult for users to enter text into edit
-            // controls which are located bottom of the screen and may not be able to view their
-            // cursor at all.
-            if (bottom > (mRenderData.screenHeight / 3)) {
-                // Center the cursor within the viewable area (not obscured by System UI).
-                mCursorOffsetScreenY = (((float) mRenderData.screenHeight - bottom) / 2.0f);
-            } else {
-                mCursorOffsetScreenY = 0.0f;
+    public void onSystemUiVisibilityChanged(SystemUiVisibilityChangedEventParameter parameter) {
+        if (parameter.systemUiVisible) {
+            mSystemUiScreenSize.set(parameter.left, parameter.top,
+                    mRenderData.screenWidth - parameter.right,
+                    mRenderData.screenHeight - parameter.bottom);
+
+            if (mAdjustViewportForSystemUi) {
+                // Adjust the cursor position to ensure it's visible when large System UI (1/3 or
+                // more of the total screen size) is displayed (typically the Soft Keyboard).
+                // Without this change, it is difficult for users to enter text into edit controls
+                // which are located bottom of the screen and may not see the cursor at all.
+                if (mSystemUiScreenSize.bottom > (mRenderData.screenHeight / 3)) {
+                    // Center the cursor within the viewable area (not obscured by System UI).
+                    mCursorOffsetScreenY = (float) parameter.bottom / 2.0f;
+                } else {
+                    mCursorOffsetScreenY = 0.0f;
+                }
+
+                // Apply the cursor offset.
+                setCursorPosition(mCursorPosition.x, mCursorPosition.y);
             }
-
-            // Apply the cursor offset.
-            setCursorPosition(mCursorPosition.x, mCursorPosition.y);
+        } else {
+            mCursorOffsetScreenY = 0.0f;
+            mSystemUiScreenSize.setEmpty();
         }
     }
 
-    /** Called to indicate that no System UI is visible. */
-    public void clearSystemUiOffsets() {
-        mCursorOffsetScreenY = 0.0f;
-        mSystemUiScreenSize.setEmpty();
-    }
-
     public void adjustViewportForSystemUi(boolean adjustViewportForSystemUi) {
         mAdjustViewportForSystemUi = adjustViewportForSystemUi;
     }