This CL adds methods to manipulate RTP header extension, particularly

content/browser/renderer_host/p2p/socket_host.h

Index: content/browser/renderer_host/p2p/socket_host.h
diff --git a/content/browser/renderer_host/p2p/socket_host.h b/content/browser/renderer_host/p2p/socket_host.h
index 40dc713fa454a89c0274dfe04bcf3a059400efb5..c0a97b76bb188d420b14fb043bf92e575f1dcdbe 100644
--- a/content/browser/renderer_host/p2p/socket_host.h
+++ b/content/browser/renderer_host/p2p/socket_host.h
@@ -49,6 +49,8 @@ class CONTENT_EXPORT P2PSocketHost {
   virtual bool SetOption(P2PSocketOption option, int value) = 0;
 
  protected:
+  void MaybeUpdateRtpSendTimeExtn(const char* data, int len);

[juberti2, 2014/02/14 01:53:27]

The naming could be clearer here - you have 3 functions that do similar things
with different (and slightly confusing) names:
MaybeURSTE works on an arbitrary packet
ParseRtpPacket works on a RTP packet
UpdateRtpSendTimeExtn works on the extension

suggest having a more unified set of names, like
MaybeUpdatePacketSendTime
MaybeUpdateRtpSendTime
UpdateRtpExtnSendTime
UpdateRtpAuthTag

[Solis, 2014/02/14 10:05:29]

I assume you mean to update the absolute sender time extension, if present.
Change name to

MaybeUpdateAbsSendTimeExtn() or
MaybeUpdateAbsoluteSenderTimeExtension()

Not sure whether the "Rtp" part is necessary or not. Maybe you could pull all
these functions into a utility namespace?

+
   friend class P2PSocketHostTcpTestBase;
 
   // TODO(mallinath) - Remove this below enum and use one defined in
@@ -93,9 +95,16 @@ class CONTENT_EXPORT P2PSocketHost {
                                 StunMessageType* type);
   static bool IsRequestOrResponse(StunMessageType type);
 
+  bool ParseRtpPacket(const char* data, int len);

[Solis, 2014/02/14 10:05:29]

I think you can remove these functions from the class - see comment in .cc.

+  int SkipPaddingBytes(const char* data, int len);
+  void UpdateRtpSendTimeExtn(const char* data, int len);
+
+  void UpdateHmac(const void* packet, int len);
+
   IPC::Sender* message_sender_;
   int id_;
   State state_;
+  int rtp_sendtime_extn_id_;
 
   DISALLOW_COPY_AND_ASSIGN(P2PSocketHost);
 };