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

Unified Diff: chrome/common/extensions/api/webrtc_logging_private.idl

Issue 1650133002: Start and stop RTC event logs from private extension API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Some comments from tommi Created 4 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/common/chrome_switches.cc ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/api/webrtc_logging_private.idl
diff --git a/chrome/common/extensions/api/webrtc_logging_private.idl b/chrome/common/extensions/api/webrtc_logging_private.idl
index 0f900f944402da663da9706a1914fad707cc5bfa..fadea0e094bc0575d2cf4642ac3e9564b111ee69 100644
--- a/chrome/common/extensions/api/webrtc_logging_private.idl
+++ b/chrome/common/extensions/api/webrtc_logging_private.idl
@@ -28,23 +28,23 @@ namespace webrtcLoggingPrivate {
long? guestProcessId;
};
- // This contains information about the result of audio debug recordings.
- dictionary AudioDebugRecordingsInfo {
+ // This contains information about the result of audio debug recordings or
+ // WebRTC event logs.
+ dictionary RecordingInfo {
// Absolute path prefix for the files with the audio debug recordings.
DOMString prefixPath;
- // Indicates if recording was stopped.
+ // Indicates if recording was stopped (either by a timed callback after the
+ // time limit has elapsed, or by a manual call).
boolean didStop;
// Indicates if recording was stopped manually through a
- // stopAudioDebugRecordings() call.
+ // stopAudioDebugRecordings() or stopWebRtcEventLogging() call.
boolean didManualStop;
};
-
callback GenericDoneCallback = void ();
- callback AudioDebugRecordingsCallback =
- void (AudioDebugRecordingsInfo info);
+ callback RecordingDoneCallback = void (RecordingInfo info);
callback UploadDoneCallback = void (UploadResult result);
interface Functions {
@@ -137,15 +137,35 @@ namespace webrtcLoggingPrivate {
// that recording has not stopped.
// If |seconds| is negative, startAudioDebugRecordings() fails.
static void startAudioDebugRecordings(RequestInfo request,
- DOMString securityOrigin,
- long seconds,
- AudioDebugRecordingsCallback callback);
+ DOMString securityOrigin,
+ long seconds,
+ RecordingDoneCallback callback);
// Stops audio debug recordings. |callback| is invoked once recording
// stops. If there is no recording in progress, stopAudioDebugRecordings()
// fails.
static void stopAudioDebugRecordings(RequestInfo request,
- DOMString securityOrigin,
- AudioDebugRecordingsCallback callback);
+ DOMString securityOrigin,
+ RecordingDoneCallback callback);
+
+ // Starts WebRTC event logging.
+ // startWebRtcEventLogging() logs the most recent events that happened
+ // before the call, and then keep logging for |seconds| seconds into the
+ // future. |callback| is invoked once the logging stops.
+ // If |seconds| is zero, the logging will continue until
+ // stopWebRtcEventLogging() is explicitly called. In this case,
+ // |callback| is invoked once recording starts and will report
+ // that recording has not stopped.
+ // If |seconds| is negative, startWebRtcEventLogging() fails.
+ static void startWebRtcEventLogging(RequestInfo request,
+ DOMString securityOrigin,
+ long seconds,
+ RecordingDoneCallback callback);
+
+ // Stops RTC event logging. |callback| is invoked once the logging stops.
+ // If there is no recording in progress, stopWebRtcEventLogging() fails.
+ static void stopWebRtcEventLogging(RequestInfo request,
+ DOMString securityOrigin,
+ RecordingDoneCallback callback);
};
};
« no previous file with comments | « chrome/common/chrome_switches.cc ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698