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

Unified Diff: chrome/browser/devtools/devtools_ui_bindings.cc

Issue 2651043006: DevTools: Remove files in the navigator when they are deleted externally (Closed)
Patch Set: fix test Created 3 years, 11 months 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/inspector/isolated-filesystem-test.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/devtools/devtools_ui_bindings.cc
diff --git a/chrome/browser/devtools/devtools_ui_bindings.cc b/chrome/browser/devtools/devtools_ui_bindings.cc
index aea197f17799bdc717b7b9366d59d0f1eba6b890..e05a3fbc8cd45fc15e1f3b9f218f2e79369c0734 100644
--- a/chrome/browser/devtools/devtools_ui_bindings.cc
+++ b/chrome/browser/devtools/devtools_ui_bindings.cc
@@ -1140,16 +1140,13 @@ void DevToolsUIBindings::FilePathsChanged(
const std::vector<std::string>& changed_paths,
const std::vector<std::string>& added_paths,
const std::vector<std::string>& removed_paths) {
- base::ListValue list;
- for (auto path : changed_paths)
- list.AppendString(path);
- for (auto path : added_paths)
- list.AppendString(path);
- for (auto path : removed_paths)
- list.AppendString(path);
-
- CallClientFunction("DevToolsAPI.fileSystemFilesChanged",
- &list, NULL, NULL);
+ base::ListValue changed, added, removed;
+ changed.AppendStrings(changed_paths);
+ added.AppendStrings(added_paths);
+ removed.AppendStrings(removed_paths);
+
+ CallClientFunction("DevToolsAPI.fileSystemFilesChangedAddedRemoved", &changed,
+ &added, &removed);
}
void DevToolsUIBindings::IndexingTotalWorkCalculated(
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/inspector/isolated-filesystem-test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698