Remove width and height property from <webview> tag.

Remove width and height property from <webview> tag.
Make the shadow <object> fill in the <webview>.
This means <browser> is now supposed to be sized using style.height/style.width css.


BUG=142721
TEST=content_browsertests.BrowserPluginHostTest*,BrowserPluginTest*
browser_tests.WebViewTest*
Updated the tests to match behavior.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=163898

[lazyboy, 2012-10-23 21:28:21]

Hey Mihai,
We're removing the width/height property from <browser> and expect to set
width/height via css. (from the status docs discussion).
We fill in the <browser> with the <object> now. This was suggested by Darin
before in the bug, but it required taking out width/height property (otherwise
ambiguous states occur).
This also requires updating the doLayout() in sample app.
Ty.

[Fady Samuel, 2012-10-24 17:16:10]

Some of these files have moved around with the <webview> change. Please update.

[Mihai Parparita -not on Chrome, 2012-10-24 17:16:53]

LGTM

https://chromiumcodereview.appspot.com/11231086/diff/2001/chrome/renderer/res...
File chrome/renderer/resources/extensions/browser_tag.js (right):

https://chromiumcodereview.appspot.com/11231086/diff/2001/chrome/renderer/res...
chrome/renderer/resources/extensions/browser_tag.js:94: var numericValue =
parseInt(value, 10);
You can now remove this.

[lazyboy, 2012-10-24 18:29:44]

Synced and updated to <webview> renaming CL.

https://chromiumcodereview.appspot.com/11231086/diff/2001/chrome/renderer/res...
File chrome/renderer/resources/extensions/browser_tag.js (right):

https://chromiumcodereview.appspot.com/11231086/diff/2001/chrome/renderer/res...
chrome/renderer/resources/extensions/browser_tag.js:94: var numericValue =
parseInt(value, 10);
On 2012/10/24 17:16:54, Mihai Parparita wrote:
> You can now remove this.

Thanks, Done.

[commit-bot: I haz the power, 2012-10-24 19:03:24]

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

[commit-bot: I haz the power, 2012-10-24 20:58:22]

Change committed as 163898