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

Side by Side Diff: ui/file_manager/integration_tests/gallery/open_image_files.js

Issue 1608143002: support animated GIF (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix a regression bug that the original image is not removed then the filter applied. Created 4 years, 11 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 'use strict'; 5 'use strict';
6 6
7 /** 7 /**
8 * Runs a test to open a single image. 8 * Runs a test to open a single image.
9 * 9 *
10 * @param {string} testVolumeName Test volume name passed to the addEntries 10 * @param {string} testVolumeName Test volume name passed to the addEntries
(...skipping 25 matching lines...) Expand all
36 bounds.outerHeight); 36 bounds.outerHeight);
37 } 37 }
38 return true; 38 return true;
39 }); 39 });
40 }); 40 });
41 }); 41 });
42 42
43 return resizedWindowPromise.then(function() { 43 return resizedWindowPromise.then(function() {
44 var rootElementPromise = 44 var rootElementPromise =
45 gallery.waitForElement(appId, '.gallery[mode="slide"]'); 45 gallery.waitForElement(appId, '.gallery[mode="slide"]');
46 var imagePromise = gallery.waitForElement(
47 appId, '.gallery .content canvas.image[width="760"]');
48 var fullImagePromsie = gallery.waitForElement( 46 var fullImagePromsie = gallery.waitForElement(
49 appId, '.gallery .content canvas.fullres[width="800"]'); 47 appId, '.gallery .image-container > .image');
50 return Promise.all([rootElementPromise, imagePromise, fullImagePromsie]). 48 return Promise.all([rootElementPromise, fullImagePromsie]).
51 then(function(args) { 49 then(function(args) {
52 chrome.test.assertEq('760', args[1].attributes.width); 50 chrome.test.assertEq(760, args[1].renderedWidth);
53 chrome.test.assertEq('570', args[1].attributes.height); 51 chrome.test.assertEq(570, args[1].renderedHeight);
hirono 2016/01/22 07:24:18 Could you add assertions for imageWidth/imageHeigh
ryoh 2016/01/25 03:29:26 Done.
54 chrome.test.assertEq('800', args[2].attributes.width);
55 chrome.test.assertEq('600', args[2].attributes.height);
56 }); 52 });
57 }); 53 });
58 }); 54 });
59 } 55 }
60 56
61 /** 57 /**
62 * Confirms that two images are loaded in thumbnail mode. This method doesn't 58 * Confirms that two images are loaded in thumbnail mode. This method doesn't
63 * care whether two images are loaded with error or not. 59 * care whether two images are loaded with error or not.
64 * 60 *
65 * @param {string} appId 61 * @param {string} appId
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 return openMultipleImages('drive', 'drive'); 151 return openMultipleImages('drive', 'drive');
156 }; 152 };
157 153
158 /** 154 /**
159 * The openMultipleImagesAndChangeToSlideMode test for Downloads. 155 * The openMultipleImagesAndChangeToSlideMode test for Downloads.
160 * @return {Promise} Promise to be fulfilled with on success. 156 * @return {Promise} Promise to be fulfilled with on success.
161 */ 157 */
162 testcase.openMultipleImagesAndChangeToSlideModeOnDownloads = function() { 158 testcase.openMultipleImagesAndChangeToSlideModeOnDownloads = function() {
163 return openMultipleImagesAndChangeToSlideMode('local', 'downloads'); 159 return openMultipleImagesAndChangeToSlideMode('local', 'downloads');
164 }; 160 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698