Implements RTP header dumping.

content/public/browser/render_process_host.h

Index: content/public/browser/render_process_host.h
diff --git a/content/public/browser/render_process_host.h b/content/public/browser/render_process_host.h
index 66a96eca5c6aca95d61c3ce68656669350f7c3fb..0d0c5367a02f76a81792607921af956537ef9e24 100644
--- a/content/public/browser/render_process_host.h
+++ b/content/public/browser/render_process_host.h
@@ -30,6 +30,14 @@ class RenderWidgetHost;
 class StoragePartition;
 struct GlobalRequestID;
 
+#if defined(ENABLE_WEBRTC)
+enum RtpDumpType {
+  RTP_DUMP_INCOMING,
+  RTP_DUMP_OUTGOING,
+  RTP_DUMP_BOTH,
+};
+#endif
+
 // Interface that represents the browser side of the browser <-> renderer
 // communication channel. There will generally be one RenderProcessHost per
 // renderer process.
@@ -205,6 +213,20 @@ class CONTENT_EXPORT RenderProcessHost : public IPC::Sender,
   // process associated with this RenderProcessHost.
   virtual void SetWebRtcLogMessageCallback(
       base::Callback<void(const std::string&)> callback) = 0;
+
+  typedef base::Callback<void(const uint8* packet_header,
+                              size_t header_length,
+                              size_t packet_length,
+                              bool incoming)> WebRtcRtpPacketCallback;
+
+  typedef base::Callback<void(RtpDumpType)> WebRtcStopRtpDumpCallback;
+
+  // Starts passing RTP packets to |packet_callback| and returns the callback
+  // used to stop dumping. This method can be called on any thread. The returned

[jam, 2014/05/16 22:03:22]

can you clarify what you mean by "called on any thread"? you mean the callback,
not StartRtpDump right?

[jiayl, 2014/05/16 22:52:43]

Improved the comment.

+  // WebRtcStopRtpDumpCallback must be called on the IO thread.
+  virtual WebRtcStopRtpDumpCallback StartRtpDump(
+      RtpDumpType,

[jam, 2014/05/16 22:03:22]

can you keep it as with booleans as before for incoming & outgoing? in the
content api we put each enum in a separate file, so making it here for these two
would bloat content/public/browser even more

[jiayl, 2014/05/16 22:52:43]

Done.

+      const WebRtcRtpPacketCallback& packet_callback) = 0;
 #endif
 
   // Tells the ResourceDispatcherHost to resume a deferred navigation without