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

Unified Diff: ui/file_manager/file_manager/foreground/js/thumbnail_loader_unittest.js

Issue 2630283002: Apply image transformation for thumbnails loaded by loadDetachedImage. (Closed)
Patch Set: Add comment. Created 3 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 side-by-side diff with in-line comments
Download patch
Index: ui/file_manager/file_manager/foreground/js/thumbnail_loader_unittest.js
diff --git a/ui/file_manager/file_manager/foreground/js/thumbnail_loader_unittest.js b/ui/file_manager/file_manager/foreground/js/thumbnail_loader_unittest.js
index 75129e461d0c8a6a990551a0e4a755e3d894a49d..f972dd1b12c70dd8436d3b4aabd1d4d326d91e09 100644
--- a/ui/file_manager/file_manager/foreground/js/thumbnail_loader_unittest.js
+++ b/ui/file_manager/file_manager/foreground/js/thumbnail_loader_unittest.js
@@ -168,3 +168,40 @@ function testLoadAsDataUrlFromExternal(callback) {
assertEquals(externalThumbnailDataUrl, result.data);
}), callback);
}
+
+function testLoadDetachedFromExifInCavnasModeThumbnailRotate(callback) {
+ ImageLoaderClient.getInstance = function() {
+ return {
+ load: function(url, callback, opt_option) {
+ // Assert that data url is passed.
+ assertTrue(/^data:/i.test(url));
+ callback({status: 'success', data: url, width: 64, height: 32});
+ }
+ };
+ };
+
+ var metadata = {
+ thumbnail: {
+ url: generateSampleImageDataUrl(64, 32),
+ transform: {
+ rotate90: 1,
+ scaleX: 1,
+ scaleY: -1,
+ }
+ }
+ };
+
+ var fileSystem = new MockFileSystem('volume-id');
+ var entry = new MockEntry(fileSystem, '/Test1.jpg');
+ var thumbnailLoader =
+ new ThumbnailLoader(entry, ThumbnailLoader.LoaderType.CANVAS, metadata);
+
+ reportPromise(
+ new Promise(function(resolve, reject) {
+ thumbnailLoader.loadDetachedImage(resolve);
+ }).then(function() {
+ var image = thumbnailLoader.getImage();
+ assertEquals(32, image.width);
+ assertEquals(64, image.height);
+ }), callback);
+}

Powered by Google App Engine
This is Rietveld 408576698