DevTools: "Delete File" in navigator should actually delete a file

third_party/WebKit/Source/devtools/front_end/persistence/FileSystemWorkspaceBinding.js

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 526 matching lines @@
      */
     function contentSet() {
       callback(this._addFile(createFilePath));
     }
   }
 
   /**
    * @override
    * @param {string} path
    */
   deleteFile(path) {

[dgozman, 2017/02/17 01:13:12]

Let's pass the UISourceCode here instead.

[lushnikov, 2017/02/17 01:57:47]

Done.

-    this._fileSystem.deleteFile(path);
+    var uiSourceCode = this.uiSourceCodeForURL(path);
+    if (!uiSourceCode)
+      return;
+    var relativePath = Persistence.FileSystemWorkspaceBinding.relativePath(uiSourceCode).join('/');

[dgozman, 2017/02/17 01:13:12]

this._filePathForUISourceCode

[lushnikov, 2017/02/17 01:57:47]

Done.

+    this._fileSystem.deleteFile(relativePath);
     this.removeUISourceCode(path);
   }
 
   /**
    * @override
    */
   remove() {
     this._fileSystemWorkspaceBinding._isolatedFileSystemManager.removeFileSystem(this._fileSystem);
   }
 
@@ skipping 23 matching lines @@
     uiSourceCode[Persistence.FileSystemWorkspaceBinding._metadata] = null;
     uiSourceCode.checkContentUpdated();
   }
 
   dispose() {
     this.removeProject();
   }
 };
 
 Persistence.FileSystemWorkspaceBinding._metadata = Symbol('FileSystemWorkspaceBinding.Metadata');