Change PacketKey to be unique for each frame packet by including

Change PacketKey to be unique for each frame packet by including
frame_ID.

PacketKey was originally generated by combining capture_time, ssrc,
and packet_id. This key may not necessarily be unique. Two or more
frames with the same cpature_time had the same key and may cause one
to be evicted from the list. This, in turn, will cause a receiver to
become blocked waiting on a frame that the PacedSender will never
attempt to send.

BUG=373560
Committed: https://crrev.com/12ede5b629ef29311dd65f8f1c6c6ae4064c3b4d
Cr-Commit-Position: refs/heads/master@{#363035}

[xjz, 2015-12-02 18:26:33]

Patchset #1 (id:1) has been deleted

[xjz, 2015-12-02 19:23:54]

Patchset #1 (id:20001) has been deleted

[xjz, 2015-12-02 19:26:29]

xjz@chromium.org changed reviewers:
+ miu@chromium.org

[miu, 2015-12-03 02:42:12]

lgtm % one nit:

https://codereview.chromium.org/1487223002/diff/40001/media/cast/net/rtp/pack...
File media/cast/net/rtp/packet_storage_unittest.cc (right):

https://codereview.chromium.org/1487223002/diff/40001/media/cast/net/rtp/pack...
media/cast/net/rtp/packet_storage_unittest.cc:27: const base::TimeTicks kTicks;
Looks like you can delete this line now.

[xjz, 2015-12-03 19:15:29]

https://codereview.chromium.org/1487223002/diff/40001/media/cast/net/rtp/pack...
File media/cast/net/rtp/packet_storage_unittest.cc (right):

https://codereview.chromium.org/1487223002/diff/40001/media/cast/net/rtp/pack...
media/cast/net/rtp/packet_storage_unittest.cc:27: const base::TimeTicks kTicks;
On 2015/12/03 02:42:12, miu wrote:
> Looks like you can delete this line now.

Done.

[xjz, 2015-12-03 19:15:41]

The CQ bit was checked by xjz@chromium.org

[xjz, 2015-12-03 19:15:41]

The patchset sent to the CQ was uploaded after l-g-t-m from miu@chromium.org
Link to the patchset: https://codereview.chromium.org/1487223002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-12-03 19:17:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1487223002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1487223002/60001

[commit-bot: I haz the power, 2015-12-03 19:50:06]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2015-12-03 19:50:58]

Description was changed from

==========
Change PacketKey to be unique for each frame packet by including
frame_ID.

PacketKey was originally generated by combining capture_time, ssrc,
and packet_id. This key may not necessarily be unique. Two or more
frames with the same cpature_time had the same key and may cause one
to be evicted from the list. This, in turn, will cause a receiver to
become blocked waiting on a frame that the PacedSender will never
attempt to send.

BUG=373560
==========

to

==========
Change PacketKey to be unique for each frame packet by including
frame_ID.

PacketKey was originally generated by combining capture_time, ssrc,
and packet_id. This key may not necessarily be unique. Two or more
frames with the same cpature_time had the same key and may cause one
to be evicted from the list. This, in turn, will cause a receiver to
become blocked waiting on a frame that the PacedSender will never
attempt to send.

BUG=373560

Committed: https://crrev.com/12ede5b629ef29311dd65f8f1c6c6ae4064c3b4d
Cr-Commit-Position: refs/heads/master@{#363035}
==========

[commit-bot: I haz the power, 2015-12-03 19:50:59]

Patchset 2 (id:??) landed as
https://crrev.com/12ede5b629ef29311dd65f8f1c6c6ae4064c3b4d
Cr-Commit-Position: refs/heads/master@{#363035}