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

Unified Diff: ui/file_manager/file_manager/background/js/duplicate_finder.js

Issue 2594523002: Postpone the content duplication check until starting import. (Closed)
Patch Set: indent fix Created 4 years 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/background/js/duplicate_finder.js
diff --git a/ui/file_manager/file_manager/background/js/duplicate_finder.js b/ui/file_manager/file_manager/background/js/duplicate_finder.js
index f6639534d11e4ee75462d4d419413e4433967107..eedaf9a91d9f5fd233bb927301f1088c6283af8b 100644
--- a/ui/file_manager/file_manager/background/js/duplicate_finder.js
+++ b/ui/file_manager/file_manager/background/js/duplicate_finder.js
@@ -198,10 +198,16 @@ importer.DispositionChecker = function(historyLoader, contentMatcher) {
};
/**
- * @param {!FileEntry} entry
- * @param {!importer.Destination} destination
- * @param {!importer.ScanMode} mode
- * @return {!Promise<!importer.Disposition>}
+ * Type for a function to return content disposition of an entry.
+ *
+ * @typedef {function(!FileEntry, !importer.Destination,
+ * !importer.ScanMode):
+ * !Promise<!importer.Disposition>}
+ */
+importer.DispositionChecker.CheckerFunction;
+
+/**
+ * @type {!importer.DispositionChecker.CheckerFunction}
*/
importer.DispositionChecker.prototype.getDisposition =
function(entry, destination, mode) {
@@ -279,9 +285,7 @@ importer.DispositionChecker.prototype.hasHistoryDuplicate_ =
* @param {!importer.HistoryLoader} historyLoader
* @param {!analytics.Tracker} tracker
*
- * @return {function(!FileEntry, !importer.Destination,
- * !importer.ScanMode):
- * !Promise<!importer.Disposition>}
+ * @return {!importer.DispositionChecker.CheckerFunction}
*/
importer.DispositionChecker.createChecker =
function(historyLoader, tracker) {

Powered by Google App Engine
This is Rietveld 408576698