Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1864)

Unified Diff: chrome/test/data/extensions/platform_apps/web_view/shim/main.js

Issue 987473002: Added the onResize and onContentResize events to GuestViewEvents. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Small fix. Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/platform_apps/web_view/shim/main.js
diff --git a/chrome/test/data/extensions/platform_apps/web_view/shim/main.js b/chrome/test/data/extensions/platform_apps/web_view/shim/main.js
index fbb4623855ec8b6c8c70fad26808dd71161f15d3..9f06f185aa3f2af9d886b9d88c7e4d8854d359a5 100644
--- a/chrome/test/data/extensions/platform_apps/web_view/shim/main.js
+++ b/chrome/test/data/extensions/platform_apps/web_view/shim/main.js
@@ -2032,6 +2032,49 @@ function testLoadDataAPI() {
document.body.appendChild(webview);
};
+// Test that the resize events fire with the correct values, and in the
+// correct order, when resizing occurs.
+function testResizeEvents() {
+ var webview = new WebView();
+ webview.src = 'about:blank';
+ webview.style.width = '600px';
+ webview.style.height = '400px';
+
+ var checkSizes = function(e) {
+ embedder.test.assertEq(e.oldWidth, 600)
+ embedder.test.assertEq(e.oldHeight, 400)
+ embedder.test.assertEq(e.newWidth, 500)
+ embedder.test.assertEq(e.newHeight, 400)
+ }
+
+ var resizeListener = function(e) {
+ webview.onresize = null;
+ webview.oncontentresize = contentResizeListener;
+
+ console.log('onresize');
+ checkSizes(e);
+ };
+
+ var contentResizeListener = function(e) {
+ webview.oncontentresize = null;
+
+ console.log('oncontentresize');
+ checkSizes(e);
+ embedder.test.succeed();
+ };
+
+ var loadstopListener = function(e) {
+ webview.removeEventListener('loadstop', loadstopListener);
+ webview.onresize = resizeListener;
+
+ console.log('Resizing <webview> width from 600px to 500px.');
+ webview.style.width = '500px';
+ }
+
+ webview.addEventListener('loadstop', loadstopListener);
+ document.body.appendChild(webview);
+};
+
embedder.test.testList = {
'testAllowTransparencyAttribute': testAllowTransparencyAttribute,
'testAutosizeHeight': testAutosizeHeight,
@@ -2108,7 +2151,8 @@ embedder.test.testList = {
'testZoomAPI' : testZoomAPI,
'testFindAPI': testFindAPI,
'testFindAPI_findupdate': testFindAPI,
- 'testLoadDataAPI': testLoadDataAPI
+ 'testLoadDataAPI': testLoadDataAPI,
+ 'testResizeEvents': testResizeEvents
};
onload = function() {

Powered by Google App Engine
This is Rietveld 408576698