BrowserPlugin: Grab the correct viewsize to report to BrowserPluginGuestDelegate

chrome/test/data/extensions/platform_apps/web_view/shim/main.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 var util = {};
 var embedder = {};
 embedder.baseGuestURL = '';
 embedder.emptyGuestURL = '';
 embedder.windowOpenGuestURL = '';
 embedder.noReferrerGuestURL = '';
@@ skipping 75 matching lines @@
 
 embedder.test.assertFalse = function(condition) {
   if (condition) {
     console.log('assertion failed: false != ' + condition);
     embedder.test.fail();
   }
 };
 
 // Tests begin.
 
+// This test verifies that a lengthy page with autosize enabled will report
+// the correct height in the sizechanged event.
+function testAutosizeHeight() {
+  var webview = document.createElement('webview');
+
+  webview.setAttribute('autosize', 'true');
+  webview.setAttribute('minwidth', 200);
+  webview.setAttribute('maxwidth', 210);
+  webview.setAttribute('minheight', 40);
+  webview.setAttribute('maxheight', 50);

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

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:53]

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

+
+  webview.addEventListener('sizechanged', function(e) {
+    embedder.test.assertEq(50, e.newHeight);
+    embedder.test.succeed();
+  });
+
+  webview.setAttribute('src',
+                       'data:text/html,' +
+                       'a<br/>b<br/>c<br/>d<br/>e<br/>f<br/>' +
+                       'a<br/>b<br/>c<br/>d<br/>e<br/>f<br/>' +
+                       'a<br/>b<br/>c<br/>d<br/>e<br/>f<br/>' +
+                       'a<br/>b<br/>c<br/>d<br/>e<br/>f<br/>' +
+                       'a<br/>b<br/>c<br/>d<br/>e<br/>f<br/>');
+  document.body.appendChild(webview);
+}
+
 // This test verifies that if a browser plugin is in autosize mode before
 // navigation then the guest starts auto-sized.
 function testAutosizeBeforeNavigation() {
   var webview = document.createElement('webview');
 
   webview.setAttribute('autosize', 'true');
   webview.setAttribute('minwidth', 200);
   webview.setAttribute('maxwidth', 210);
   webview.setAttribute('minheight', 100);
   webview.setAttribute('maxheight', 110);
@@ skipping 1619 matching lines @@
     });
     wv.find("dog");
     wv.find("cat");
     wv.find("dog");
   });
 
   document.body.appendChild(webview);
 };
 
 embedder.test.testList = {
+  'testAutosizeHeight': testAutosizeHeight,
   'testAutosizeAfterNavigation': testAutosizeAfterNavigation,
   'testAutosizeBeforeNavigation': testAutosizeBeforeNavigation,
   'testAutosizeRemoveAttributes': testAutosizeRemoveAttributes,
   'testAutosizeWithPartialAttributes': testAutosizeWithPartialAttributes,
   'testAPIMethodExistence': testAPIMethodExistence,
   'testChromeExtensionURL': testChromeExtensionURL,
   'testChromeExtensionRelativePath': testChromeExtensionRelativePath,
   'testDisplayNoneWebviewLoad': testDisplayNoneWebviewLoad,
   'testDisplayNoneWebviewRemoveChild': testDisplayNoneWebviewRemoveChild,
   'testInlineScriptFromAccessibleResources':
@@ skipping 55 matching lines @@
   'testFindAPI': testFindAPI,
   'testFindAPI_findupdate': testFindAPI
 };
 
 onload = function() {
   chrome.test.getConfig(function(config) {
     embedder.setUp_(config);
     chrome.test.sendMessage("Launched");
   });
 };