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

Unified Diff: content/shell/renderer/test_runner/test_interfaces.cc

Issue 1079843002: DevTools: allow storing devtools preferences on the browser side. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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
Index: content/shell/renderer/test_runner/test_interfaces.cc
diff --git a/content/shell/renderer/test_runner/test_interfaces.cc b/content/shell/renderer/test_runner/test_interfaces.cc
index 6b88763ad931c258581e2bfcbab6950858896708..247b0524fcd7b336ed016a78d0c8ce8ebff73fd2 100644
--- a/content/shell/renderer/test_runner/test_interfaces.cc
+++ b/content/shell/renderer/test_runner/test_interfaces.cc
@@ -118,22 +118,8 @@ void TestInterfaces::ConfigureForTestWithURL(const blink::WebURL& test_url,
test_runner_->ClearDevToolsLocalStorage();
if (spec.find("/inspector/") != std::string::npos) {
// Subfolder name determines default panel to open.
- std::string settings = "";
std::string test_path = spec.substr(spec.find("/inspector/") + 11);
- size_t slash_index = test_path.find("/");
- std::string test_path_setting = base::StringPrintf(
- "\"testPath\":\"\\\"%s\\\"\"", spec.c_str());
-
- // TODO(pfeldman): remove once migrated to testPath.
- std::string last_active_panel;
- if (slash_index != std::string::npos) {
- last_active_panel = base::StringPrintf(
- ",\"lastActivePanel\":\"\\\"%s\\\"\"",
- test_path.substr(0, slash_index).c_str());
- }
-
- test_runner_->ShowDevTools(base::StringPrintf("{%s%s}",
- test_path_setting.c_str(), last_active_panel.c_str()), std::string());
+ test_runner_->ShowDevTools(spec.c_str(), std::string());
dgozman 2015/04/13 13:37:48 Either pass |test_path.c_str()| or remove |test_pa
pfeldman 2015/04/13 16:08:10 Done.
}
if (spec.find("/viewsource/") != std::string::npos) {
test_runner_->setShouldEnableViewSource(true);

Powered by Google App Engine
This is Rietveld 408576698