Reland after fix: Improve <webview> autosize:

Reland after fix: Improve <webview> autosize:
a. Expand/shrink <webview> element when 'sizechange' event fires, to
match with the new view size (in shim).
b. For SW mode, fix a bug where damage buffer would remain smaller than
the view size and would result in crash. Added test for this case.

BUG=173238, 282116
Test=WebViewTest.AutoSize.*, <webview>.autosize=true now should autosize
webview container within the constraints
(minwidth/maxwidth/minheight/maxheight).

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221023

[lazyboy, 2013-08-31 01:27:00]

Ran 200 times on Mac this time, also a trick to expose the race the code was
hitting before was to DelayTask the BrowserPlugin::OnUpdateRect handling code.

See diff patch set 1 and 2 for the fixes:
1. Changed web_view.js min and max calculation to match BrowserPlugin's min and
max calculation.
2. Restricted the dirty flag setting to not turn it on when
UpdateGuestAutoSizeState is called from OnUpdateRect.
3. Changed the test (Shim_TestAutosizeWithPartial*) itself a bit. It is possible
and (expected) that some sizechanged event will be dropped. If the renderer
tries to say change size A->B->A, the browser might only see A and A, missing
the A->B and B->A transition (and we didn't change the size to B at all).
The test was a bit aggressive expecting to see B.

[lazyboy, 2013-09-03 18:03:51]

+sadrul as fady is in vacation.

[sadrul, 2013-09-03 18:04:20]

A small nit, otherwise lgtm

https://chromiumcodereview.appspot.com/23503023/diff/3001/chrome/test/data/ex...
File chrome/test/data/extensions/platform_apps/web_view/shim/main.js (right):

https://chromiumcodereview.appspot.com/23503023/diff/3001/chrome/test/data/ex...
chrome/test/data/extensions/platform_apps/web_view/shim/main.js:186:
embedder.test.assertTrue(e.newWidth <= 700);
can 500 and 700 be written as webview.minWidth/webview.maxWidth instead?

[Fady Samuel, 2013-09-03 18:05:35]

lgtm

[lazyboy, 2013-09-03 18:47:35]

https://chromiumcodereview.appspot.com/23503023/diff/3001/chrome/test/data/ex...
File chrome/test/data/extensions/platform_apps/web_view/shim/main.js (right):

https://chromiumcodereview.appspot.com/23503023/diff/3001/chrome/test/data/ex...
chrome/test/data/extensions/platform_apps/web_view/shim/main.js:186:
embedder.test.assertTrue(e.newWidth <= 700);
On 2013/09/03 18:04:20, sadrul wrote:
> can 500 and 700 be written as webview.minWidth/webview.maxWidth instead?

Done.

[commit-bot: I haz the power, 2013-09-03 18:48:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/lazyboy@chromium.org/23503023/21001

[commit-bot: I haz the power, 2013-09-03 21:12:13]

Change committed as 221023