DevTools: [Persistence] guard against multiple UISourceCodes with similar URL

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

Index: third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js
diff --git a/third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js b/third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js
index 0a0bd178b62cd8b9dd369500c53d07d7b7e8d650..817cad606a3a75ebd61f5e8dff0f8f5f1712d059 100644
--- a/third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js
+++ b/third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js
@@ -32,6 +32,9 @@ Persistence.Persistence = class extends Common.Object {
    * @param {!Persistence.PersistenceBinding} binding
    */
   _onBindingCreated(binding) {
+    // FIXME(lushnikov): workspace should contain only one UISourceCode per URL.

[dgozman, 2016/12/01 18:29:05]

TODO

+    if (binding.network[Persistence.Persistence._binding] || binding.fileSystem[Persistence.Persistence._binding])

[dgozman, 2016/12/01 18:29:06]

I think we should move this check to both mappings instead.

[lushnikov, 2016/12/02 01:54:55]

Since Persistence instance is nicely isolated from mappings - this feels like a
good place by itself. wdyt?

+      return;
     if (binding.network.isDirty()) {
       Common.console.log(
           Common.UIString('%s can not be persisted to file system due to unsaved changes.', binding.network.name()));