Adds StartRtpDump and StopRtpDump methods to webrtcLoggingPrivate API.

chrome/common/extensions/api/webrtc_logging_private.idl

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 b07aed44cd5f5e5cba33be29b6560608f75945ab..db4c94764df433b674bb3ec19ba56b1607232409 100644
--- a/chrome/common/extensions/api/webrtc_logging_private.idl
+++ b/chrome/common/extensions/api/webrtc_logging_private.idl
@@ -57,7 +57,8 @@
                      DOMString securityOrigin,
                      GenericDoneCallback callback);
 
-    // Uploads the log. Logging must be stopped before this function is called.
+    // Uploads the log and the RTP dumps, if they exist. Logging and RTP dumping
+    // must be stopped before this function is called.
     static void upload(long tabId,
                        DOMString securityOrigin,
                        UploadDoneCallback callback);
@@ -66,5 +67,22 @@
     static void discard(long tabId,
                         DOMString securityOrigin,
                         GenericDoneCallback callback);
+
+    // Starts RTP dumping. If it has already been started for this render
+    // process, the call will be ignored.
+    static void startRtpDump(long tabId,
+                             DOMString securityOrigin,
+                             boolean incoming,
+                             boolean outgoing,
+                             GenericDoneCallback callback);
+
+    // Stops RTP dumping. After stop has finished, the dumps will be
+    // uploaded with the log if upload is called. Otherwise, the dumps will be
+    // discarded.
+    static void stopRtpDump(long tabId,
+                            DOMString securityOrigin,
+                            boolean incoming,

[Henrik Grunell, 2014/05/05 14:15:09]

Is |incoming| and |outgoing| necessary? Maybe simply stop whatever has started?

[jiayl, 2014/05/05 16:26:54]

The existing UI for the GTP allows the user to check/uncheck incoming/outgoing
dump separately. So I think we might need this flexibility.

+                            boolean outgoing,
+                            GenericDoneCallback callback);
   };
 };