Reland attempt 2: Improve <webview> autosize:

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 152 matching lines @@
   document.body.appendChild(webview);
 }
 
 // This test verifies that autosize works when some of the parameters are unset.
 function testAutosizeWithPartialAttributes() {
   window.console.log('testAutosizeWithPartialAttributes');
   var webview = document.createElement('webview');
 
   var step = 1;
   var sizeChangeHandler = function(e) {
+    window.console.log('sizeChangeHandler, new: ' +
+                       e.newWidth + ' X ' + e.newHeight);
     switch (step) {
       case 1:
         // Expect 300x200.
         embedder.test.assertEq(300, e.newWidth);
         embedder.test.assertEq(200, e.newHeight);
 
-        // Remove an autosize attribute and verify that it causes relayout.
-        webview.minwidth = null;
+        // Change the min size to cause a relayout.
+        webview.minwidth = 500;
         break;
       case 2:
-        // Expect 640x?
-        embedder.test.assertEq(640, e.newWidth);
+        embedder.test.assertTrue(e.newWidth >= webview.minwidth);
+        embedder.test.assertTrue(e.newWidth <= webview.maxwidth);
 
         // Tests when minwidth > maxwidth, minwidth = maxwidth.
-        webview.maxheight = null;
+        // i.e. minwidth is essentially 700.
         webview.minwidth = 800;
-        webview.minheight = 800;
         break;
       case 3:
-        // Expect 700X480
+        // Expect 700X?
         embedder.test.assertEq(700, e.newWidth);
-        embedder.test.assertEq(480, e.newHeight);
-
-        // Remove maxwidth and make sure the size returns to plugin size.
-        webview.removeAttribute('maxwidth');
-        break;
-      case 4:
-        // Expect original size, 640x480.
-        embedder.test.assertEq(640, e.newWidth);
-        embedder.test.assertEq(480, e.newHeight);
+        embedder.test.assertTrue(e.newHeight >= 200);
+        embedder.test.assertTrue(e.newHeight <= 600);
 
         embedder.test.succeed();
         break;
       default:
         window.console.log('Unexpected sizechanged event, step = ' + step);
         embedder.test.fail();
         break;
     }
 
     ++step;
@@ skipping 24 matching lines @@
   var sizeChangeHandler = function(e) {
     switch (step) {
       case 1:
         // This is the sizechanged event for autosize.
 
         // Remove attributes.
         webview.removeAttribute('minwidth');
         webview.removeAttribute('maxwidth');
         webview.removeAttribute('minheight');
         webview.removeAttribute('maxheight');
+        webview.removeAttribute('autosize');
+
+        // We'd get one more sizechanged event after we turn off
+        // autosize.
+        webview.style.width = '500px';
+        webview.style.height = '500px';
         break;
       case 2:
-        // Expect plugin to return to its original size.
-        embedder.test.assertEq(640, e.newWidth);
-        embedder.test.assertEq(480, e.newHeight);
-
         embedder.test.succeed();
         break;
-      default:
-        window.console.log('Unexpected sizechanged event, step = ' + step);
-        embedder.test.fail();
-        break;
     }
 
     ++step;
   };
 
   webview.addEventListener('loadstop', function(e) {
     webview.minwidth = 300;
     webview.maxwidth = 700;
     webview.minheight = 600;
     webview.maxheight = 400;
@@ skipping 772 matching lines @@
   'testRemoveWebviewAfterNavigation': testRemoveWebviewAfterNavigation,
   'testResizeWebviewResizesContent': testResizeWebviewResizesContent
 };
 
 onload = function() {
   chrome.test.getConfig(function(config) {
     embedder.setUp_(config);
     chrome.test.sendMessage("Launched");
   });
 };