Add support for size checking selection based scans.

ui/file_manager/file_manager/background/js/media_scanner.js

Index: ui/file_manager/file_manager/background/js/media_scanner.js
diff --git a/ui/file_manager/file_manager/background/js/media_scanner.js b/ui/file_manager/file_manager/background/js/media_scanner.js
index 895c7fa03e32de81d2f64365d061bd047c1e8f78..84c0b21f3b3c9228fa1da66608463c381a7ee71c 100644
--- a/ui/file_manager/file_manager/background/js/media_scanner.js
+++ b/ui/file_manager/file_manager/background/js/media_scanner.js
@@ -135,20 +135,22 @@ importer.DefaultMediaScanner.prototype.scan = function(entries) {
   }
 
   var scanResult = this.createScanResult_();
-  var watcher = this.watcherFactory_(function() {
-    scanResult.invalidateScan();
-    this.observers_.forEach(
-        /** @param {!importer.ScanObserver} observer */
-        function(observer) {
-          observer(importer.ScanEvent.INVALIDATED, scanResult);
-        });
-  }.bind(this));
+  var watcher = this.watcherFactory_(
+      /** @this {importer.DefaultMediaScanner} */
+      function() {
+        scanResult.invalidateScan();
+        this.observers_.forEach(
+            /** @param {!importer.ScanObserver} observer */
+            function(observer) {
+              observer(importer.ScanEvent.INVALIDATED, scanResult);
+            });
+      }.bind(this));
   var scanPromises = entries.map(
       this.scanEntry_.bind(this, scanResult, watcher));
 
   Promise.all(scanPromises)
-      .then(scanResult.resolveScan.bind(scanResult))
-      .catch(scanResult.rejectScan.bind(scanResult));
+      .then(scanResult.resolve)
+      .catch(scanResult.reject);
 
   scanResult.whenFinal()
       .then(
@@ -279,33 +281,23 @@ importer.DefaultScanResult = function(hashGenerator, historyLoader) {
    */
   this.lastScanActivity_ = this.scanStarted_;
 
-  /** @private {boolean} */
-  this.settled_ = false;
+  /** @private {!importer.Resolver.<!importer.ScanResult>} */
+  this.resolver_ = new importer.Resolver();
+};
 
-  /** @type {function()} */
-  this.resolveScan;
+/** @struct */
+importer.DefaultScanResult.prototype = {
 
-  /** @type {function(*)} */
-  this.rejectScan;
+  /** @return {function()} */

[Ben Kwa, 2015/01/30 23:13:09]

I think this is the reason your refactoring addressed the type issue: the type
of the resolve function being passed to the .then call above has gone back to
{function()}, whereas in your intermediate code it was... something else.

[Steve McKay, 2015/01/30 23:18:02]

But the issue was only when I had individually selected files...not for all
scans. Seems like if the issue was related to the original code...then it would
have affected *all* scans.

+  get resolve() { return this.resolver_.resolve.bind(null, this); },
 
-  /** @private {!Promise.<!importer.ScanResult>} */
-  this.finishedPromise_ = new Promise(
-      function(resolve, reject) {
-        this.resolveScan = function() {
-          this.settled_ = true;
-          resolve(this);
-        }.bind(this);
-
-        this.rejectScan = function() {
-          this.settled_ = true;
-          reject();
-        };
-      }.bind(this));
+  /** @return {function(*=)} */
+  get reject() { return this.resolver_.reject; }
 };
 
 /** @override */
 importer.DefaultScanResult.prototype.isFinal = function() {
-  return this.settled_;
+  return this.resolver_.settled;
 };
 
 importer.DefaultScanResult.prototype.isInvalidated = function() {
@@ -329,7 +321,7 @@ importer.DefaultScanResult.prototype.getScanDurationMs = function() {
 
 /** @override */
 importer.DefaultScanResult.prototype.whenFinal = function() {
-  return this.finishedPromise_;
+  return this.resolver_.promise;
 };
 
 /**