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

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

Issue 23499003: Improve <webview> autosize: (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add UMA for autosize. 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 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
177 embedder.test.assertEq(200, e.newHeight); 177 embedder.test.assertEq(200, e.newHeight);
178 178
179 // Remove an autosize attribute and verify that it causes relayout. 179 // Remove an autosize attribute and verify that it causes relayout.
180 webview.minwidth = null; 180 webview.minwidth = null;
181 break; 181 break;
182 case 2: 182 case 2:
183 // Expect 640x? 183 // Expect 640x?
184 embedder.test.assertEq(640, e.newWidth); 184 embedder.test.assertEq(640, e.newWidth);
185 185
186 // Tests when minwidth > maxwidth, minwidth = maxwidth. 186 // Tests when minwidth > maxwidth, minwidth = maxwidth.
187 webview.maxheight = null;
188 webview.minwidth = 800; 187 webview.minwidth = 800;
189 webview.minheight = 800;
190 break; 188 break;
191 case 3: 189 case 3:
192 // Expect 700X480 190 // Expect 700X?
193 embedder.test.assertEq(700, e.newWidth); 191 embedder.test.assertEq(700, e.newWidth);
194 embedder.test.assertEq(480, e.newHeight); 192 embedder.test.assertTrue(e.newHeight >= 200);
193 embedder.test.assertTrue(e.newHeight <= 600);
195 194
196 // Remove maxwidth and make sure the size returns to plugin size. 195 // Remove maxwidth and make sure the size returns to plugin size.
197 webview.removeAttribute('maxwidth'); 196 webview.removeAttribute('maxwidth');
198 break; 197 break;
199 case 4: 198 case 4:
200 // Expect original size, 640x480. 199 // Expect original size, 640x480.
201 embedder.test.assertEq(640, e.newWidth); 200 embedder.test.assertEq(640, e.newWidth);
202 embedder.test.assertEq(480, e.newHeight); 201 embedder.test.assertTrue(e.newHeight >= 200);
202 embedder.test.assertTrue(e.newHeight <= 600);
203 203
204 embedder.test.succeed(); 204 embedder.test.succeed();
205 break; 205 break;
206 default: 206 default:
207 window.console.log('Unexpected sizechanged event, step = ' + step); 207 window.console.log('Unexpected sizechanged event, step = ' + step);
208 embedder.test.fail(); 208 embedder.test.fail();
209 break; 209 break;
210 } 210 }
211 211
212 ++step; 212 ++step;
(...skipping 715 matching lines...) Expand 10 before | Expand all | Expand 10 after
928 'testRemoveWebviewAfterNavigation': testRemoveWebviewAfterNavigation, 928 'testRemoveWebviewAfterNavigation': testRemoveWebviewAfterNavigation,
929 'testResizeWebviewResizesContent': testResizeWebviewResizesContent 929 'testResizeWebviewResizesContent': testResizeWebviewResizesContent
930 }; 930 };
931 931
932 onload = function() { 932 onload = function() {
933 chrome.test.getConfig(function(config) { 933 chrome.test.getConfig(function(config) {
934 embedder.setUp_(config); 934 embedder.setUp_(config);
935 chrome.test.sendMessage("Launched"); 935 chrome.test.sendMessage("Launched");
936 }); 936 });
937 }; 937 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698