Remove rejected promise of PiexLoader which is not caught.

ui/file_manager/image_loader/piex_loader.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @typedef {{
  *   fulfill: function(PiexLoaderResponse):undefined,
  *   reject: function(string):undefined}
  * }}
  */
@@ skipping 47 matching lines @@
  * @struct
  */
 function PiexLoader() {
   /**
    * @private {!Element}
    * @const
    */
   this.naclModule_ = document.createElement('embed');
 
   /**
-   * @private {!Promise}
+   * @private {!Promise<boolean>}
    * @const
    */
   this.naclPromise_ = new Promise(function(fulfill) {
     chrome.fileManagerPrivate.isPiexLoaderEnabled(fulfill);
   }).then(function(enabled) {
     if (!enabled)
-      return Promise.reject('PiexLoader is not enabled for chromium build.');
+      return false;
     return new Promise(function(fulfill, reject) {
       var embed = this.naclModule_;
       embed.setAttribute('type', 'application/x-pnacl');
       // The extension nmf is not allowed to load. We uses .nmf.js instead.
       embed.setAttribute('src', '/piex/piex.nmf.txt');
       embed.width = 0;
       embed.height = 0;
 
       // The <EMBED> element is wrapped inside a <DIV>, which has both a 'load'
       // and a 'message' event listener attached.  This wrapping method is used
       // instead of attaching the event listeners directly to the <EMBED>
       // element to ensure that the listeners are active before the NaCl module
       // 'load' event fires.
       var listenerContainer = document.createElement('div');
       listenerContainer.appendChild(embed);
       listenerContainer.addEventListener('load', fulfill, true);
       listenerContainer.addEventListener(
           'message', this.onMessage_.bind(this), true);
       listenerContainer.addEventListener('error', function() {
         reject(embed['lastError']);
       }.bind(this), true);
       listenerContainer.addEventListener('crash', function() {
         reject('PiexLoader crashed.');
       }.bind(this), true);
       listenerContainer.style.height = '0px';
       document.body.appendChild(listenerContainer);
+      return true;

[oka, 2017/02/07 17:01:33]

wondering why it's not fulfill(true).

[hirono, 2017/02/09 03:52:55]

Yes, the line is wrong. Removed this and updated #91. Thanks!

     }.bind(this));
-  }.bind(this));
+  }.bind(this)).catch(function (error) {
+    console.error(error);
+    return false;
+  });
 
   /**
    * @private {!Object<number, PiexRequestCallbacks>}
    * @const
    */
   this.requests_ = {};
 
   /**
    * @private {number}
    */
@@ skipping 14 matching lines @@
   }
   delete this.requests_[id];
 };
 
 /**
  * Starts to load RAW image.
  * @param {string} url
  * @return {!Promise<!PiexLoaderResponse>}
  */
 PiexLoader.prototype.load = function(url) {
-  return this.naclPromise_.then(function() {
+  return this.naclPromise_.then(function(loaded) {
+    if (!loaded)
+      return Promise.reject('Piex is not loaded');
     var message = {
       id: this.requestIdCount_++,
       name: 'loadThumbnail',
       url: url
     };
     this.naclModule_.postMessage(message);
     return new Promise(function(fulfill, reject) {
       this.requests_[message.id] = {fulfill: fulfill, reject: reject};
-    }.bind(this));
+    }.bind(this)).catch(function(error) {
+      console.error('PiexLoaderError: ', error);
+      return Promise.reject(error);
+    });
   }.bind(this));
 };