Use ScaleToEnclosingRect when converting the viewport to window.

Use ScaleToEnclosingRect when converting the viewport to window.

This guarantees that the viewport is always contained in the window.

BUG=none

Committed: https://crrev.com/88fd07f9af5311a6e85502aedca5ce94c9ddfe23
Cr-Commit-Position: refs/heads/master@{#414452}

[lfg, 2016-08-17 15:56:28]

lfg@chromium.org changed reviewers:
+ oshima@chromium.org

[lfg, 2016-08-17 15:56:28]

+oshima, please take a look.

[oshima, 2016-08-23 20:58:54]

lgtm

[lfg, 2016-08-24 14:44:41]

lfg@chromium.org changed reviewers:
+ piman@chromium.org

[lfg, 2016-08-24 14:44:41]

+piman for OWNERS review.

[piman, 2016-08-24 17:31:33]

lgtm

[lfg, 2016-08-24 17:57:18]

The CQ bit was checked by lfg@chromium.org

[commit-bot: I haz the power, 2016-08-24 17:58:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-24 19:44:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-24 19:44:52]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[lfg, 2016-08-25 15:48:24]

The CQ bit was checked by lfg@chromium.org

[commit-bot: I haz the power, 2016-08-25 15:48:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 16:23:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 16:26:05]

Description was changed from

==========
Use ScaleToEnclosingRect when converting the viewport to window.

This guarantees that the viewport is always contained in the window.

BUG=none
==========

to

==========
Use ScaleToEnclosingRect when converting the viewport to window.

This guarantees that the viewport is always contained in the window.

BUG=none

Committed: https://crrev.com/88fd07f9af5311a6e85502aedca5ce94c9ddfe23
Cr-Commit-Position: refs/heads/master@{#414452}
==========

[commit-bot: I haz the power, 2016-08-25 16:26:06]

Patchset 1 (id:??) landed as
https://crrev.com/88fd07f9af5311a6e85502aedca5ce94c9ddfe23
Cr-Commit-Position: refs/heads/master@{#414452}

[johnme, 2016-08-25 17:23:23]

On 2016/08/25 16:26:06, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/88fd07f9af5311a6e85502aedca5ce94c9ddfe23
> Cr-Commit-Position: refs/heads/master@{#414452}

It seems this may have broken
virtual/scalefactor200withzoom/fast/hidpi/static/popup-menu-appearance.html
on
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/6...

[lfg, 2016-08-25 17:44:07]

On 2016/08/25 17:23:23, johnme wrote:
> On 2016/08/25 16:26:06, commit-bot: I haz the power wrote:
> > Patchset 1 (id:??) landed as
> > https://crrev.com/88fd07f9af5311a6e85502aedca5ce94c9ddfe23
> > Cr-Commit-Position: refs/heads/master@{#414452}
> 
> It seems this may have broken
> virtual/scalefactor200withzoom/fast/hidpi/static/popup-menu-appearance.html
> on
>
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/6...

Just checked, it seems wrong. I'll revert for now.

[lfg, 2016-08-25 17:44:40]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2279873002/ by lfg@chromium.org.

The reason for reverting is: This seems to have broken
virtual/scalefactor200withzoom/fast/hidpi/static/popup-menu-appearance.html..

[johnme, 2016-08-25 17:58:14]

Thanks for reverting. It also seems to have broken
virtual/scalefactor200withzoom/fast/hidpi/static/popup-menu-appearance.html
Across these bots: WebKit Linux WebKit Linux Leak WebKit Linux Trusty WebKit
Win10 WebKit Win7