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

Unified Diff: chrome/test/chromedriver/logging_unittest.cc

Issue 353063005: [ChromeDriver] Subscribe PerformanceLogger to CommandListener interface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 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: chrome/test/chromedriver/logging_unittest.cc
diff --git a/chrome/test/chromedriver/logging_unittest.cc b/chrome/test/chromedriver/logging_unittest.cc
index 7912b6360eff526784122fce4ed6cf0f74ec77cf..0a9a3193027e7a4874d0031fd9b2e4fb916e0767 100644
--- a/chrome/test/chromedriver/logging_unittest.cc
+++ b/chrome/test/chromedriver/logging_unittest.cc
@@ -101,12 +101,15 @@ TEST(Logging, CreatePerformanceLog) {
capabilities.logging_prefs["performance"] = Log::kInfo;
capabilities.logging_prefs["browser"] = Log::kInfo;
- ScopedVector<DevToolsEventListener> listeners;
+ ScopedVector<DevToolsEventListener> devtools_listeners;
ScopedVector<WebDriverLog> logs;
- Status status = CreateLogs(capabilities, &logs, &listeners);
+ std::list<CommandListener*> command_listeners;
+ Status status = CreateLogs(capabilities, &logs, &devtools_listeners,
+ &command_listeners);
ASSERT_TRUE(status.IsOk());
ASSERT_EQ(2u, logs.size());
- ASSERT_EQ(2u, listeners.size());
+ ASSERT_EQ(2u, devtools_listeners.size());
+ ASSERT_EQ(1u, command_listeners.size());
ASSERT_EQ("performance", logs[0]->type());
ASSERT_EQ("browser", logs[1]->type());
}
@@ -115,24 +118,30 @@ TEST(Logging, IgnoreUnknownLogType) {
Capabilities capabilities;
capabilities.logging_prefs["gaga"] = Log::kInfo;
- ScopedVector<DevToolsEventListener> listeners;
+ ScopedVector<DevToolsEventListener> devtools_listeners;
ScopedVector<WebDriverLog> logs;
- Status status = CreateLogs(capabilities, &logs, &listeners);
+ std::list<CommandListener*> command_listeners;
+ Status status = CreateLogs(capabilities, &logs, &devtools_listeners,
+ &command_listeners);
EXPECT_TRUE(status.IsOk());
ASSERT_EQ(1u, logs.size());
- ASSERT_EQ(1u, listeners.size());
+ ASSERT_EQ(1u, devtools_listeners.size());
+ ASSERT_EQ(0u, command_listeners.size());
ASSERT_EQ("browser", logs[0]->type());
}
TEST(Logging, DefaultLogs) {
Capabilities capabilities;
- ScopedVector<DevToolsEventListener> listeners;
+ ScopedVector<DevToolsEventListener> devtools_listeners;
ScopedVector<WebDriverLog> logs;
- Status status = CreateLogs(capabilities, &logs, &listeners);
+ std::list<CommandListener*> command_listeners;
+ Status status = CreateLogs(capabilities, &logs, &devtools_listeners,
+ &command_listeners);
EXPECT_TRUE(status.IsOk());
ASSERT_EQ(1u, logs.size());
- ASSERT_EQ(1u, listeners.size());
+ ASSERT_EQ(1u, devtools_listeners.size());
+ ASSERT_EQ(0u, command_listeners.size());
ASSERT_EQ("browser", logs[0]->type());
}

Powered by Google App Engine
This is Rietveld 408576698