DevTools: restore selection and scrollposition between network and filesystem

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @extends {WebInspector.Object}
  * @param {!WebInspector.Workspace} workspace
  * @param {!WebInspector.BreakpointManager} breakpointManager
  * @param {!WebInspector.FileSystemMapping} fileSystemMapping
@@ skipping 20 matching lines @@
     BindingCreated: Symbol("BindingCreated"),
     BindingRemoved: Symbol("BindingRemoved")
 }
 
 WebInspector.Persistence.prototype = {
     /**
      * @param {!WebInspector.PersistenceBinding} binding
      */
     _onBindingCreated: function(binding)
     {
-        if (binding.network.isDirty() || binding.fileSystem.isDirty()) {
+        if (binding.network.isDirty()) {
             WebInspector.console.log(WebInspector.UIString("%s can not be persisted to file system due to unsaved changes.", binding.network.name()));
             return;
         }
+        if (binding.fileSystem.isDirty())
+            binding.network.setWorkingCopy(binding.fileSystem.workingCopy());
+
         binding.network[WebInspector.Persistence._binding] = binding;
         binding.fileSystem[WebInspector.Persistence._binding] = binding;
 
         binding.fileSystem.forceLoadOnCheckContent();
 
         binding.network.addEventListener(WebInspector.UISourceCode.Events.WorkingCopyCommitted, this._onWorkingCopyCommitted, this);
         binding.fileSystem.addEventListener(WebInspector.UISourceCode.Events.WorkingCopyCommitted, this._onWorkingCopyCommitted, this);
 
         this._addFilePathBindingPrefixes(binding.fileSystem.url());
 
         this._moveBreakpoints(binding.fileSystem, binding.network);
         this.dispatchEventToListeners(WebInspector.Persistence.Events.BindingCreated, binding);
     },
 
     /**
      * @param {!WebInspector.PersistenceBinding} binding
      */
     _onBindingRemoved: function(binding)
     {
+        if (binding.network.isDirty())
+            binding.fileSystem.setWorkingCopy(binding.network.workingCopy());

[dgozman, 2016/10/24 18:14:50]

Can we test working copy as well?

[lushnikov, 2016/10/25 00:45:26]

Done.

+
         binding.network[WebInspector.Persistence._binding] = null;
         binding.fileSystem[WebInspector.Persistence._binding] = null;
 
         binding.network.removeEventListener(WebInspector.UISourceCode.Events.WorkingCopyCommitted, this._onWorkingCopyCommitted, this);
         binding.fileSystem.removeEventListener(WebInspector.UISourceCode.Events.WorkingCopyCommitted, this._onWorkingCopyCommitted, this);
 
         this._removeFilePathBindingPrefixes(binding.fileSystem.url());
 
         this._copyBreakpoints(binding.network, binding.fileSystem);
         this.dispatchEventToListeners(WebInspector.Persistence.Events.BindingRemoved, binding);
@@ skipping 162 matching lines @@
  * @param {!WebInspector.UISourceCode} fileSystem
  */
 WebInspector.PersistenceBinding = function(network, fileSystem)
 {
     this.network = network;
     this.fileSystem = fileSystem;
 }
 
 /** @type {!WebInspector.Persistence} */
 WebInspector.persistence;