In RtpPacket do not keep pointer to RtpHeaderExtensionMap

In RtpPacket do not keep pointer to RtpHeaderExtensionMap

Having that dependency require user of RtpPacket to ensure
RtpHeaderExtensionMap always outlive packet and that RtpPacket's access
to RtpHeaderExtensionMap is properly syncrhonized.
Dropping this dependencies make use of RtpPacket less error-prone.

BUG=webrtc:5261
Review-Url: https://codereview.webrtc.org/2576653003
Cr-Commit-Position: refs/heads/master@{#15653}
Committed: https://chromium.googlesource.com/external/webrtc/+/70f39a30e96082a49809850422c8fe5275accdf8

[danilchap, 2016-12-14 16:21:03]

The CQ bit was checked by danilchap@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-14 16:21:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[danilchap, 2016-12-14 16:22:14]

Description was changed from

==========
In RtpPacket do not keep pointer to RtpHeaderExtensionMap

Having that dependency require user of RtpPacket to ensure
RtpHeaderExtensionMap always outlive packet and that RtpPacket's access
to RtpHeaderExtensionMap is properly syncrhonized.
Dropping this dependencies make use of RtpPacket less error-prone.

BUG=None
==========

to

==========
In RtpPacket do not keep pointer to RtpHeaderExtensionMap

Having that dependency require user of RtpPacket to ensure
RtpHeaderExtensionMap always outlive packet and that RtpPacket's access
to RtpHeaderExtensionMap is properly syncrhonized.
Dropping this dependencies make use of RtpPacket less error-prone.

BUG=webrtc:5261
==========

[danilchap, 2016-12-14 16:25:27]

danilchap@webrtc.org changed reviewers:
+ brandtr@webrtc.org

[commit-bot: I haz the power, 2016-12-14 16:42:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-14 16:42:59]

Dry run: Try jobs failed on following builders:
  linux_msan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/16054)

[brandtr, 2016-12-16 12:30:43]

lgtm

Thanks for changing!

[danilchap, 2016-12-16 13:22:28]

The CQ bit was checked by danilchap@webrtc.org

[commit-bot: I haz the power, 2016-12-16 13:22:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-12-16 13:48:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481894548152460,
"parent_rev": "7e70fe2b961bdac80fec324907b441dcb354542b", "commit_rev":
"70f39a30e96082a49809850422c8fe5275accdf8"}

[commit-bot: I haz the power, 2016-12-16 13:48:21]

Description was changed from

==========
In RtpPacket do not keep pointer to RtpHeaderExtensionMap

Having that dependency require user of RtpPacket to ensure
RtpHeaderExtensionMap always outlive packet and that RtpPacket's access
to RtpHeaderExtensionMap is properly syncrhonized.
Dropping this dependencies make use of RtpPacket less error-prone.

BUG=webrtc:5261
==========

to

==========
In RtpPacket do not keep pointer to RtpHeaderExtensionMap

Having that dependency require user of RtpPacket to ensure
RtpHeaderExtensionMap always outlive packet and that RtpPacket's access
to RtpHeaderExtensionMap is properly syncrhonized.
Dropping this dependencies make use of RtpPacket less error-prone.

BUG=webrtc:5261

Review-Url: https://codereview.webrtc.org/2576653003
Cr-Commit-Position: refs/heads/master@{#15653}
Committed:
https://chromium.googlesource.com/external/webrtc/+/70f39a30e96082a4980985042...
==========

[commit-bot: I haz the power, 2016-12-16 13:48:22]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/70f39a30e96082a4980985042...