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

Unified Diff: ui/file_manager/gallery/js/image_editor/image_view.js

Issue 971173002: Gallery: Start to use new metadata model in Gallery. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test. Created 5 years, 10 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
« no previous file with comments | « ui/file_manager/gallery/js/image_editor/image_util.js ('k') | ui/file_manager/gallery/js/mosaic_mode.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/gallery/js/image_editor/image_view.js
diff --git a/ui/file_manager/gallery/js/image_editor/image_view.js b/ui/file_manager/gallery/js/image_editor/image_view.js
index 6bea3ef85351e4da3f8eda63145f3f0df9539137..9cd4989831f1b6820110efcefbbe3b3c5f0a90ba 100644
--- a/ui/file_manager/gallery/js/image_editor/image_view.js
+++ b/ui/file_manager/gallery/js/image_editor/image_view.js
@@ -7,11 +7,12 @@
*
* @param {!HTMLElement} container The container element.
* @param {!Viewport} viewport The viewport.
+ * @param {!MetadataModel} metadataModel
* @constructor
* @extends {ImageBuffer.Overlay}
* @struct
*/
-function ImageView(container, viewport) {
+function ImageView(container, viewport, metadataModel) {
ImageBuffer.Overlay.call(this);
this.container_ = container;
@@ -27,10 +28,12 @@ function ImageView(container, viewport) {
this.contentGeneration_ = 0;
this.displayedContentGeneration_ = 0;
- this.imageLoader_ = new ImageUtil.ImageLoader(this.document_);
+ this.imageLoader_ =
+ new ImageUtil.ImageLoader(this.document_, metadataModel);
// We have a separate image loader for prefetch which does not get cancelled
// when the selection changes.
- this.prefetchLoader_ = new ImageUtil.ImageLoader(this.document_);
+ this.prefetchLoader_ =
+ new ImageUtil.ImageLoader(this.document_, metadataModel);
this.contentCallbacks_ = [];
@@ -412,26 +415,16 @@ ImageView.prototype.load =
*/
function displayThumbnail(loadType, canvas) {
if (canvas) {
- var width = null;
- var height = null;
- if (metadata.media) {
- width = metadata.media.width;
- height = metadata.media.height;
- }
- // If metadata.external.present is true, the image data is loaded directly
- // from local cache, whose size may be out of sync with the drive
- // metadata.
- if (metadata.external && !metadata.external.present) {
- width = metadata.external.imageWidth;
- height = metadata.external.imageHeight;
- }
+ var width = item.getMetadataItem().imageWidth;
+ var height = item.getMetadataItem().imageHeight;
self.replace(
canvas,
effect,
width,
height,
true /* preview */);
- if (displayCallback) displayCallback();
+ if (displayCallback)
+ displayCallback();
}
loadMainImage(loadType, entry, !!canvas,
(effect && canvas) ? effect.getSafeInterval() : 0);
« no previous file with comments | « ui/file_manager/gallery/js/image_editor/image_util.js ('k') | ui/file_manager/gallery/js/mosaic_mode.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698