Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(397)

Unified Diff: third_party/WebKit/LayoutTests/inspector/sources/debugger/file-system-project-mapping.html

Issue 2506463002: DevTools: remove Bindings.NetworkMapping.addMapping/removeMapping methods (Closed)
Patch Set: rebaseline Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/inspector/sources/debugger/file-system-project-mapping.html
diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger/file-system-project-mapping.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger/file-system-project-mapping.html
index e5ed9dcd9338880e8f0ad600cea4de4c69b3588d..5b625fb49e77ec17ab9f67b77f9a8c3a51bf2935 100644
--- a/third_party/WebKit/LayoutTests/inspector/sources/debugger/file-system-project-mapping.html
+++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger/file-system-project-mapping.html
@@ -14,7 +14,6 @@ function test()
var defaultScriptMapping;
var persistence;
var fileSystemProjectId = Bindings.FileSystemWorkspaceBinding.projectId("file:///var/www");
- Bindings.networkMapping.dispose();
function createWorkspaceWithTarget()
{
@@ -60,7 +59,9 @@ function test()
var uiSourceCode = InspectorTest.testWorkspace.uiSourceCode(fileSystemProjectId, "file:///var/www/html/foo.js");
networkUISourceCode = InspectorTest.testWorkspace.uiSourceCode(Bindings.NetworkProject.projectId(target, target.resourceTreeModel.mainFrame, false), "http://localhost/html/foo.js");
InspectorTest.addResult("Adding mapping between network and file system resources.");
- InspectorTest.testNetworkMapping.addMapping(networkUISourceCode, uiSourceCode);
+
+ var fileSystemPath = Bindings.FileSystemWorkspaceBinding.fileSystemPath(uiSourceCode.project().id());
+ Workspace.fileSystemMapping.addMappingForResource(networkUISourceCode.url(), fileSystemPath, uiSourceCode.url());
var setting = JSON.stringify(Workspace.fileSystemMapping._fileSystemMappingSetting.get());
InspectorTest.addResult("Emulate reloading inspector.");
@@ -80,7 +81,7 @@ function test()
InspectorTest.addResult("Removing mapping between network and file system resources.");
var uiSourceCode = InspectorTest.testWorkspace.uiSourceCode(fileSystemProjectId, "file:///var/www/html/foo.js");
- InspectorTest.testNetworkMapping.removeMapping(uiSourceCode);
+ Workspace.fileSystemMapping.removeMappingForURL(uiSourceCode.url());
InspectorTest.addResult("Emulate reloading inspector.");
fs.reportRemoved();

Powered by Google App Engine
This is Rietveld 408576698