Issue 6621

lib/ioStorage.js

Index: lib/ioStorage.js
diff --git a/lib/io.js b/lib/ioStorage.js
similarity index 81%
rename from lib/io.js
rename to lib/ioStorage.js
index a919c5dbe6d83e8e4d092b00a85247c0e4fb5620..7e73e8657d209ff118341f06fab7ed4ec43c811f 100644
--- a/lib/io.js
+++ b/lib/ioStorage.js
@@ -81,20 +81,6 @@ exports.IO =
     return saveFile(fileName, data);
   },
 
-  /**
-   * Copies a file.
-   * @param {string} fromFile
-   *    Name of the file to be copied
-   * @param {string} toFile
-   *    Name of the file to be written, will be overwritten if exists
-   * @return {Promise}
-   *    Promise to be resolved or rejected once the operation is completed
-   */
-  copyFile(fromFile, toFile)
-  {
-    return loadFile(fromFile).then(entry => saveFile(toFile, entry.content));
-  },
-
   /**
    * Renames a file.
    * @param {string} fromFile
@@ -111,18 +97,6 @@ exports.IO =
       .then(() => this.removeFile(fromFile));
   },
 
-  /**
-   * Removes a file.
-   * @param {string} fileName
-   *    Name of the file to be removed
-   * @return {Promise}
-   *    Promise to be resolved or rejected once the operation is completed
-   */
-  removeFile(fileName)
-  {
-    return browser.storage.local.remove(fileToKey(fileName));
-  },

[Sebastian Noack, 2018/06/01 19:15:46]

Well spotted! I confirmed that copyFile() and removeFile() are no longer used
(by other code). So removing them here (and not implementing them in ioIndexedDB
in the first place) is the right thing to do.

However, removeFile() is internally called by renameFile(). So you have to adapt
the code there to call browser.storage.local.remove() directly.

[piscoi.georgiana, 2018/06/05 07:15:56]

Done.

-
   /**
    * Retrieves file metadata.
    * @param {string} fileName