Implements RTP header dumping.

content/public/browser/render_process_host.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_
 
 #include "base/basictypes.h"
 #include "base/id_map.h"
 #include "base/process/kill.h"
@@ skipping 187 matching lines @@
 
 #if defined(ENABLE_WEBRTC)
   virtual void EnableAecDump(const base::FilePath& file) = 0;
   virtual void DisableAecDump() = 0;
 
   // When set, |callback| receives log messages regarding, for example, media
   // devices (webcams, mics, etc) that were initially requested in the render
   // 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;
+  virtual void SetWebRtcRtpPacketCallback(
+      const WebRtcRtpPacketCallback& callback) = 0;
+
+  virtual void StartRtpDump(bool incoming, bool outgoing) = 0;

[jam, 2014/05/14 23:59:35]

can you simplify this api by:
1) removing teh SetWebRtcRtpPacketCallback method and instead passing that
callback to StartRtpDump

2) removing StopRtpDump by having StartRtpDump return a callback that is used to
stop?

3) remove OnRtpPacke from RenderProcessHostImpl and instead have the webrtc code
use the callback directly instead of going through RPHImpl

[jiayl, 2014/05/15 23:06:22]

Great idea! Done.

+  virtual void StopRtpDump(bool incoming, bool outgoing) = 0;
 #endif
 
   // Tells the ResourceDispatcherHost to resume a deferred navigation without
   // transferring it to a new renderer process.
   virtual void ResumeDeferredNavigation(const GlobalRequestID& request_id) = 0;
 
   // Notifies the renderer that the timezone configuration of the system might
   // have changed.
   virtual void NotifyTimezoneChange() = 0;
 
@@ skipping 48 matching lines @@
   static void SetMaxRendererProcessCount(size_t count);
 
   // Returns the current max number of renderer processes used by the content
   // module.
   static size_t GetMaxRendererProcessCount();
 };
 
 }  // namespace content.
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_