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

Unified Diff: chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc

Issue 1530863002: Allow audio debug recordings through a private API extension. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/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());
+}

Powered by Google App Engine
This is Rietveld 408576698