DevTools: move FileSystemWorkspaceBinding under persistence/

DevTools: move FileSystemWorkspaceBinding under persistence/

This patch moves FileSystemWorkspaceBinding under persistence/ module,
as planned before.

Ideally, the creation of FileSystemWorkspaceBinding should be moved
inside persistence as well; however, tests push back to this change,
thus the change hasn't been made.

BUG=649837
R=dgozman

Committed: https://crrev.com/8237ed8ad1d23a7f95c5ebafb9692fe79754180e
Cr-Commit-Position: refs/heads/master@{#433370}

[lushnikov, 2016-11-18 02:58:40]

please, take a look

[lushnikov, 2016-11-18 02:58:46]

The CQ bit was checked by lushnikov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-18 03:00:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 04:47:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-18 04:47:51]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-11-18 19:32:58]

lgtm

[lushnikov, 2016-11-18 19:47:44]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-11-18 19:48:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 20:26:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-18 20:26:42]

Failed to apply patch for
third_party/WebKit/Source/devtools/front_end/main/Main.js:
While running git apply --index -p1;
  error: patch failed:
third_party/WebKit/Source/devtools/front_end/main/Main.js:186
  error: third_party/WebKit/Source/devtools/front_end/main/Main.js: patch does
not apply

Patch:       third_party/WebKit/Source/devtools/front_end/main/Main.js
Index: third_party/WebKit/Source/devtools/front_end/main/Main.js
diff --git a/third_party/WebKit/Source/devtools/front_end/main/Main.js
b/third_party/WebKit/Source/devtools/front_end/main/Main.js
index
de094030b0c7a2d8408d55e2c69cc5fd92023570..2b4c09cd08d4c25ece2632ab410bac7fb00ce9d3
100644
--- a/third_party/WebKit/Source/devtools/front_end/main/Main.js
+++ b/third_party/WebKit/Source/devtools/front_end/main/Main.js
@@ -186,18 +186,16 @@ Main.Main = class {
     Common.formatterWorkerPool = new Common.FormatterWorkerPool();
     Workspace.fileSystemMapping = new
Workspace.FileSystemMapping(Workspace.isolatedFileSystemManager);
 
-    var fileSystemWorkspaceBinding =
-        new
Bindings.FileSystemWorkspaceBinding(Workspace.isolatedFileSystemManager,
Workspace.workspace);
     Main.networkProjectManager = new
Bindings.NetworkProjectManager(SDK.targetManager, Workspace.workspace);
     Bindings.presentationConsoleMessageHelper = new
Bindings.PresentationConsoleMessageHelper(Workspace.workspace);
-    Bindings.cssWorkspaceBinding =
-        new Bindings.CSSWorkspaceBinding(SDK.targetManager,
Workspace.workspace);
-    Bindings.debuggerWorkspaceBinding =
-        new Bindings.DebuggerWorkspaceBinding(SDK.targetManager,
Workspace.workspace);
+    Bindings.cssWorkspaceBinding = new
Bindings.CSSWorkspaceBinding(SDK.targetManager, Workspace.workspace);
+    Bindings.debuggerWorkspaceBinding = new
Bindings.DebuggerWorkspaceBinding(SDK.targetManager, Workspace.workspace);
     Bindings.breakpointManager =
         new Bindings.BreakpointManager(null, Workspace.workspace,
SDK.targetManager, Bindings.debuggerWorkspaceBinding);
     Extensions.extensionServer = new Extensions.ExtensionServer();
 
+    var fileSystemWorkspaceBinding =
+        new
Persistence.FileSystemWorkspaceBinding(Workspace.isolatedFileSystemManager,
Workspace.workspace);
     Persistence.persistence =
         new Persistence.Persistence(Workspace.workspace,
Bindings.breakpointManager, Workspace.fileSystemMapping);

[lushnikov, 2016-11-18 23:46:38]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2016-11-18 23:46:38]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2515583002/#ps20001
(title: "rebaseline")

[commit-bot: I haz the power, 2016-11-18 23:47:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-19 01:31:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-19 01:33:46]

Description was changed from

==========
DevTools: move FileSystemWorkspaceBinding under persistence/

This patch moves FileSystemWorkspaceBinding under persistence/ module,
as planned before.

Ideally, the creation of FileSystemWorkspaceBinding should be moved
inside persistence as well; however, tests push back to this change,
thus the change hasn't been made.

BUG=649837
R=dgozman
==========

to

==========
DevTools: move FileSystemWorkspaceBinding under persistence/

This patch moves FileSystemWorkspaceBinding under persistence/ module,
as planned before.

Ideally, the creation of FileSystemWorkspaceBinding should be moved
inside persistence as well; however, tests push back to this change,
thus the change hasn't been made.

BUG=649837
R=dgozman

Committed: https://crrev.com/8237ed8ad1d23a7f95c5ebafb9692fe79754180e
Cr-Commit-Position: refs/heads/master@{#433370}
==========

[commit-bot: I haz the power, 2016-11-19 01:33:48]

Patchset 2 (id:??) landed as
https://crrev.com/8237ed8ad1d23a7f95c5ebafb9692fe79754180e
Cr-Commit-Position: refs/heads/master@{#433370}