Index: chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc |
diff --git a/chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc b/chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc |
index cb5ab5e97f233d208432a7facc410d5051310087..2729934e86f1a03b415c5beb216f2e6310b1f314 100644 |
--- a/chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc |
+++ b/chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc |
@@ -11,6 +11,7 @@ |
#include "chrome/browser/extensions/extension_tab_util.h" |
#include "chrome/browser/media/webrtc_log_uploader.h" |
#include "chrome/browser/ui/tabs/tab_strip_model.h" |
+#include "chrome/common/chrome_switches.h" |
#include "content/public/browser/notification_service.h" |
#include "content/public/test/test_utils.h" |
#include "extensions/common/test_util.h" |
@@ -25,6 +26,8 @@ using extensions::WebrtcLoggingPrivateStopRtpDumpFunction; |
using extensions::WebrtcLoggingPrivateStoreFunction; |
using extensions::WebrtcLoggingPrivateUploadFunction; |
using extensions::WebrtcLoggingPrivateUploadStoredFunction; |
+using extensions::WebrtcLoggingPrivateStartAecDumpFunction; |
+using extensions::WebrtcLoggingPrivateStopAecDumpFunction; |
namespace utils = extension_function_test_utils; |
@@ -54,7 +57,6 @@ void InitializeTestMetaData(base::ListValue* parameters) { |
class WebrtcLoggingPrivateApiTest : public ExtensionApiTest { |
protected: |
- |
void SetUp() override { |
ExtensionApiTest::SetUp(); |
extension_ = extensions::test_util::CreateEmptyExtension(); |
@@ -158,6 +160,22 @@ class WebrtcLoggingPrivateApiTest : public ExtensionApiTest { |
return RunFunction<WebrtcLoggingPrivateUploadStoredFunction>(params, true); |
} |
+ bool StartAecDump(int seconds) { |
+ base::ListValue params; |
+ params.AppendInteger(seconds); |
+ return RunFunction<WebrtcLoggingPrivateStartAecDumpFunction>(params, true); |
+ } |
+ |
+ bool StopAecDump() { |
+ return RunNoArgsFunction<WebrtcLoggingPrivateStopAecDumpFunction>(false); |
+ } |
+ |
+ bool StartAecDumpNoFlag(int seconds) { |
+ base::ListValue params; |
+ params.AppendInteger(seconds); |
+ return RunFunction<WebrtcLoggingPrivateStartAecDumpFunction>(params, false); |
+ } |
+ |
private: |
scoped_refptr<Extension> extension_; |
}; |
@@ -393,3 +411,12 @@ IN_PROC_BROWSER_TEST_F(WebrtcLoggingPrivateApiTest, |
buffer_override.multipart().find(kTestLoggingUrl)); |
} |
+IN_PROC_BROWSER_TEST_F(WebrtcLoggingPrivateApiTest, TestStartStopAecDump) { |
+ // TODO(guidou): These tests are missing verification of the actual AEC dump |
+ // data. This will be fixed with a separate browser test. |
+ // See crbug.com/569957. |
+ base::CommandLine::ForCurrentProcess()->AppendSwitch( |
+ switches::kEnableAecDumps); |
+ ASSERT_TRUE(StartAecDump(0)); |
+ ASSERT_TRUE(StopAecDump()); |
+} |