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

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

Issue 13145003: Rewrite std::string("") to std::string(), Linux edition. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Ugh Created 7 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
« no previous file with comments | « chrome/browser/devtools/devtools_file_helper.cc ('k') | chrome/browser/download/download_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/devtools/devtools_sanity_browsertest.cc
diff --git a/chrome/browser/devtools/devtools_sanity_browsertest.cc b/chrome/browser/devtools/devtools_sanity_browsertest.cc
index 6a8b30ceaf5d6d750fdbd003734095e4eb78da90..4f832f7b771e6018a1572d394aaa9c8d7c11b03a 100644
--- a/chrome/browser/devtools/devtools_sanity_browsertest.cc
+++ b/chrome/browser/devtools/devtools_sanity_browsertest.cc
@@ -441,7 +441,7 @@ IN_PROC_BROWSER_TEST_F(
IN_PROC_BROWSER_TEST_F(DevToolsExtensionTest,
TestDevToolsExtensionAPI) {
LoadExtension("devtools_extension");
- RunTest("waitForTestResultsInConsole", "");
+ RunTest("waitForTestResultsInConsole", std::string());
}
// Tests that chrome.devtools extension can communicate with background page
@@ -449,7 +449,7 @@ IN_PROC_BROWSER_TEST_F(DevToolsExtensionTest,
IN_PROC_BROWSER_TEST_F(DevToolsExtensionTest,
TestDevToolsExtensionMessaging) {
LoadExtension("devtools_messaging");
- RunTest("waitForTestResultsInConsole", "");
+ RunTest("waitForTestResultsInConsole", std::string());
}
// Tests that chrome.experimental.devtools extension is correctly exposed
@@ -457,7 +457,7 @@ IN_PROC_BROWSER_TEST_F(DevToolsExtensionTest,
IN_PROC_BROWSER_TEST_F(DevToolsExperimentalExtensionTest,
TestDevToolsExperimentalExtensionAPI) {
LoadExtension("devtools_experimental");
- RunTest("waitForTestResultsInConsole", "");
+ RunTest("waitForTestResultsInConsole", std::string());
}
// Tests that a content script is in the scripts list.
@@ -478,7 +478,7 @@ IN_PROC_BROWSER_TEST_F(DevToolsExtensionTest,
IN_PROC_BROWSER_TEST_F(DevToolsSanityTest,
MAYBE_TestRendererProcessNativeMemorySize) {
- RunTest("testRendererProcessNativeMemorySize", "");
+ RunTest("testRendererProcessNativeMemorySize", std::string());
}
// Tests that scripts are not duplicated after Scripts Panel switch.
« no previous file with comments | « chrome/browser/devtools/devtools_file_helper.cc ('k') | chrome/browser/download/download_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698