mediaview: Change the contract of watcher callbacks.

components/arc/common/file_system.mojom

Index: components/arc/common/file_system.mojom
diff --git a/components/arc/common/file_system.mojom b/components/arc/common/file_system.mojom
index 0fc8ecb91d458b1752b57a05f4d4ccb6019d9fba..96dab12f0e62711766beaabfc2462c0d08e9ff22 100644
--- a/components/arc/common/file_system.mojom
+++ b/components/arc/common/file_system.mojom
@@ -37,9 +37,9 @@ enum ChangeType {
   CHANGED = 0,
 
   // Indicates that the watched document itself was deleted.
-  // When OnDocumentChanged() is called with this change type, the corresponding
-  // watcher has been already uninstalled and the host should not call
-  // RemoveWatcher().
+  // Even if OnDocumentChanged() is called with this change type, the
+  // corresponding watcher is not uninstalled automatically. The host must
+  // call RemoveWatcher() to clean up an orphaned watcher.
   DELETED = 1,
 };
 
@@ -47,9 +47,7 @@ enum ChangeType {
 // Next method ID: 1
 interface FileSystemHost {
   // Called when a watched document was changed.
-  // |type| describes the type of change made to the document. If |type| is
-  // DELETED, the watcher has been already removed and the host should not call
-  // RemoveWatcher().
+  // |type| describes the type of change made to the document.
   [MinVersion=3] OnDocumentChanged@0(int64 watcher_id, ChangeType type);
 };
 
@@ -117,9 +115,7 @@ interface FileSystemInstance {
   // with the watcher ID. Whether OnDocumentChanged() is called or not after
   // this method is called and before this method returns is undefined.
   //
-  // It fails if the specified watcher does not exist. Note that a watcher
-  // can be automatically uninstalled when a watched document is deleted
-  // (notified by OnDocumentChanged() with |type| = DELETED).
+  // It fails if the specified watcher does not exist.
   [MinVersion=3] RemoveWatcher@7(int64 watcher_id) => (bool success);
 
   // Requests MediaProvider to scan specified files.