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

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

Issue 23503023: Reland after fix: Improve <webview> autosize: (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments from sadrul@ Created 7 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 var util = {}; 5 var util = {};
6 var embedder = {}; 6 var embedder = {};
7 embedder.baseGuestURL = ''; 7 embedder.baseGuestURL = '';
8 embedder.emptyGuestURL = ''; 8 embedder.emptyGuestURL = '';
9 embedder.windowOpenGuestURL = ''; 9 embedder.windowOpenGuestURL = '';
10 embedder.noReferrerGuestURL = ''; 10 embedder.noReferrerGuestURL = '';
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
163 document.body.appendChild(webview); 163 document.body.appendChild(webview);
164 } 164 }
165 165
166 // This test verifies that autosize works when some of the parameters are unset. 166 // This test verifies that autosize works when some of the parameters are unset.
167 function testAutosizeWithPartialAttributes() { 167 function testAutosizeWithPartialAttributes() {
168 window.console.log('testAutosizeWithPartialAttributes'); 168 window.console.log('testAutosizeWithPartialAttributes');
169 var webview = document.createElement('webview'); 169 var webview = document.createElement('webview');
170 170
171 var step = 1; 171 var step = 1;
172 var sizeChangeHandler = function(e) { 172 var sizeChangeHandler = function(e) {
173 window.console.log('sizeChangeHandler, new: ' +
174 e.newWidth + ' X ' + e.newHeight);
173 switch (step) { 175 switch (step) {
174 case 1: 176 case 1:
175 // Expect 300x200. 177 // Expect 300x200.
176 embedder.test.assertEq(300, e.newWidth); 178 embedder.test.assertEq(300, e.newWidth);
177 embedder.test.assertEq(200, e.newHeight); 179 embedder.test.assertEq(200, e.newHeight);
178 180
179 // Remove an autosize attribute and verify that it causes relayout. 181 // Change the min size to cause a relayout.
180 webview.minwidth = null; 182 webview.minwidth = 500;
181 break; 183 break;
182 case 2: 184 case 2:
183 // Expect 640x? 185 embedder.test.assertTrue(e.newWidth >= webview.minwidth);
184 embedder.test.assertEq(640, e.newWidth); 186 embedder.test.assertTrue(e.newWidth <= webview.maxwidth);
185 187
186 // Tests when minwidth > maxwidth, minwidth = maxwidth. 188 // Tests when minwidth > maxwidth, minwidth = maxwidth.
187 webview.maxheight = null; 189 // i.e. minwidth is essentially 700.
188 webview.minwidth = 800; 190 webview.minwidth = 800;
189 webview.minheight = 800;
190 break; 191 break;
191 case 3: 192 case 3:
192 // Expect 700X480 193 // Expect 700X?
193 embedder.test.assertEq(700, e.newWidth); 194 embedder.test.assertEq(700, e.newWidth);
194 embedder.test.assertEq(480, e.newHeight); 195 embedder.test.assertTrue(e.newHeight >= 200);
195 196 embedder.test.assertTrue(e.newHeight <= 600);
196 // Remove maxwidth and make sure the size returns to plugin size.
197 webview.removeAttribute('maxwidth');
198 break;
199 case 4:
200 // Expect original size, 640x480.
201 embedder.test.assertEq(640, e.newWidth);
202 embedder.test.assertEq(480, e.newHeight);
203 197
204 embedder.test.succeed(); 198 embedder.test.succeed();
205 break; 199 break;
206 default: 200 default:
207 window.console.log('Unexpected sizechanged event, step = ' + step); 201 window.console.log('Unexpected sizechanged event, step = ' + step);
208 embedder.test.fail(); 202 embedder.test.fail();
209 break; 203 break;
210 } 204 }
211 205
212 ++step; 206 ++step;
(...skipping 809 matching lines...) Expand 10 before | Expand all | Expand 10 after
1022 'testRemoveWebviewAfterNavigation': testRemoveWebviewAfterNavigation, 1016 'testRemoveWebviewAfterNavigation': testRemoveWebviewAfterNavigation,
1023 'testResizeWebviewResizesContent': testResizeWebviewResizesContent 1017 'testResizeWebviewResizesContent': testResizeWebviewResizesContent
1024 }; 1018 };
1025 1019
1026 onload = function() { 1020 onload = function() {
1027 chrome.test.getConfig(function(config) { 1021 chrome.test.getConfig(function(config) {
1028 embedder.setUp_(config); 1022 embedder.setUp_(config);
1029 chrome.test.sendMessage("Launched"); 1023 chrome.test.sendMessage("Launched");
1030 }); 1024 });
1031 }; 1025 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698