Updating initial zoom level for remote connections in Android Client.

Updating initial zoom level for remote connections in Android Client.

This change updates the method used for sizing the remote desktop image
to the Android device's screen.  The previous method was to scale down
the remote image so that the entire image would fit.  This worked great
for similarly sized displays but not so great for high resolution or
multi-monitor displays.

The method I use now is to scale up the remote image (if needed) such
that there isn't any empty space on the screen.  I do this by
calculating the width and height ratios and scaling the image by the
larger of the two.  This ensures that the local screen is filled with
the image of the remote display and is similar to the behavior that
existed prior to my last change.  The viewport is then translated to the
upper left of the screen for the default starting position.

Once the image is resized and positioned, I update starting coordinates
for the cursor so it is placed in the center of the viewport area.

BUG=618385
Committed: https://crrev.com/b144cbc55799ed4e4105ea21f030d51e9897a715
Cr-Commit-Position: refs/heads/master@{#399103}

[joedow, 2016-06-09 23:35:11]

joedow@chromium.org changed reviewers:
+ lambroslambrou@chromium.org

[joedow, 2016-06-09 23:35:12]

PTAL!

Thanks,
Joe

[Lambros, 2016-06-10 01:01:51]

lgtm

https://codereview.chromium.org/2058543002/diff/20001/remoting/android/java/s...
File remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java
(right):

https://codereview.chromium.org/2058543002/diff/20001/remoting/android/java/s...
remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java:105: // If
the image is smaller than the screen in either dimension, then we want to scale
it
Blank line before comment please.

[joedow, 2016-06-10 03:46:16]

Thanks!

https://codereview.chromium.org/2058543002/diff/20001/remoting/android/java/s...
File remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java
(right):

https://codereview.chromium.org/2058543002/diff/20001/remoting/android/java/s...
remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java:105: // If
the image is smaller than the screen in either dimension, then we want to scale
it
On 2016/06/10 01:01:51, Lambros wrote:
> Blank line before comment please.

Done.

[joedow, 2016-06-10 03:47:12]

The CQ bit was checked by joedow@chromium.org

[joedow, 2016-06-10 03:47:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
lambroslambrou@chromium.org
Link to the patchset: https://codereview.chromium.org/2058543002/#ps40001
(title: "Addressing CR Feedback")

[commit-bot: I haz the power, 2016-06-10 03:47:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2058543002/40001

[commit-bot: I haz the power, 2016-06-10 05:01:58]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-10 05:03:46]

Description was changed from

==========
Updating initial zoom level for remote connections in Android Client.

This change updates the method used for sizing the remote desktop image
to the Android device's screen.  The previous method was to scale down
the remote image so that the entire image would fit.  This worked great
for similarly sized displays but not so great for high resolution or
multi-monitor displays.

The method I use now is to scale up the remote image (if needed) such
that there isn't any empty space on the screen.  I do this by
calculating the width and height ratios and scaling the image by the
larger of the two.  This ensures that the local screen is filled with
the image of the remote display and is similar to the behavior that
existed prior to my last change.  The viewport is then translated to the
upper left of the screen for the default starting position.

Once the image is resized and positioned, I update starting coordinates
for the cursor so it is placed in the center of the viewport area.

BUG=618385
==========

to

==========
Updating initial zoom level for remote connections in Android Client.

This change updates the method used for sizing the remote desktop image
to the Android device's screen.  The previous method was to scale down
the remote image so that the entire image would fit.  This worked great
for similarly sized displays but not so great for high resolution or
multi-monitor displays.

The method I use now is to scale up the remote image (if needed) such
that there isn't any empty space on the screen.  I do this by
calculating the width and height ratios and scaling the image by the
larger of the two.  This ensures that the local screen is filled with
the image of the remote display and is similar to the behavior that
existed prior to my last change.  The viewport is then translated to the
upper left of the screen for the default starting position.

Once the image is resized and positioned, I update starting coordinates
for the cursor so it is placed in the center of the viewport area.

BUG=618385

Committed: https://crrev.com/b144cbc55799ed4e4105ea21f030d51e9897a715
Cr-Commit-Position: refs/heads/master@{#399103}
==========

[commit-bot: I haz the power, 2016-06-10 05:03:47]

Patchset 3 (id:??) landed as
https://crrev.com/b144cbc55799ed4e4105ea21f030d51e9897a715
Cr-Commit-Position: refs/heads/master@{#399103}