BrowserPlugin: Grab the correct viewsize to report to BrowserPluginGuestDelegate

BrowserPlugin: Grab the correct viewsize to report to BrowserPluginGuestDelegate

This size is used to determine when autosize has resized the guest.

BUG=273089
Test=Launch chrome with --enable-embedded-extension-options, Open up chrome://extensions, click on Options for Goolge SPDY Proxy VPN and it shows a scrollbar.

Committed: https://crrev.com/478927e20934f4c809047b7816876d1ec38e3bf0
Cr-Commit-Position: refs/heads/master@{#293611}

[Fady Samuel, 2014-09-05 19:52:26]

fsamuel@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2014-09-05 19:52:56]

LG can you test it?

[Fady Samuel, 2014-09-05 21:08:26]

PTAL Dana.

[danakj, 2014-09-05 21:13:19]

https://codereview.chromium.org/544313002/diff/40001/chrome/test/data/extensi...
File chrome/test/data/extensions/platform_apps/web_view/shim/main.js (right):

https://codereview.chromium.org/544313002/diff/40001/chrome/test/data/extensi...
chrome/test/data/extensions/platform_apps/web_view/shim/main.js:105:
webview.setAttribute('maxheight', 50);
This makes a layer tree in the child that is taller than 50 and/or wider than
210? Can you verify/show this in the test somehow?

[Fady Samuel, 2014-09-05 21:31:54]

PTAL

https://codereview.chromium.org/544313002/diff/40001/chrome/test/data/extensi...
File chrome/test/data/extensions/platform_apps/web_view/shim/main.js (right):

https://codereview.chromium.org/544313002/diff/40001/chrome/test/data/extensi...
chrome/test/data/extensions/platform_apps/web_view/shim/main.js:105:
webview.setAttribute('maxheight', 50);
On 2014/09/05 21:13:19, danakj wrote:
> This makes a layer tree in the child that is taller than 50 and/or wider than
> 210? Can you verify/show this in the test somehow?

This test is focused on height. I've added steps to ensure that height changes
when the maxheight changes.

[danakj, 2014-09-05 21:32:59]

LGTM

[Fady Samuel, 2014-09-05 21:35:25]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2014-09-05 21:36:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fsamuel@chromium.org/544313002/70001

[commit-bot: I haz the power, 2014-09-06 03:38:49]

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

[commit-bot: I haz the power, 2014-09-06 03:38:50]

Exceeded time limit waiting for builds to trigger.

[Fady Samuel, 2014-09-06 03:39:31]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2014-09-06 03:40:48]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fsamuel@chromium.org/544313002/70001

[commit-bot: I haz the power, 2014-09-06 09:42:38]

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

[commit-bot: I haz the power, 2014-09-06 09:42:39]

Exceeded time limit waiting for builds to trigger.

[Fady Samuel, 2014-09-06 11:32:18]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2014-09-06 11:33:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fsamuel@chromium.org/544313002/70001

[commit-bot: I haz the power, 2014-09-06 13:17:26]

Committed patchset #5 (id:70001) as eda3dccca081e4923348dfa50ccde8c26f090c65

[commit-bot: I haz the power, 2014-09-10 03:42:23]

Patchset 5 (id:??) landed as
https://crrev.com/478927e20934f4c809047b7816876d1ec38e3bf0
Cr-Commit-Position: refs/heads/master@{#293611}