Break backwards traversal loop if we have looped around all packet in the PacketBuffer for H264 fra…

webrtc/modules/video_coding/packet_buffer.cc

Index: webrtc/modules/video_coding/packet_buffer.cc
diff --git a/webrtc/modules/video_coding/packet_buffer.cc b/webrtc/modules/video_coding/packet_buffer.cc
index 3b0d09f042efc77ec7cb77a92b33877d0493237a..585152bbe917e4350623895bb1bcdefae18d091c 100644
--- a/webrtc/modules/video_coding/packet_buffer.cc
+++ b/webrtc/modules/video_coding/packet_buffer.cc
@@ -234,10 +234,10 @@ std::vector<std::unique_ptr<RtpFrameObject>> PacketBuffer::FindFrames(
         // See: https://bugs.chromium.org/p/webrtc/issues/detail?id=7106
         //
         // Since we ignore the |frame_begin| flag of the inserted packets
-        // we check that |start_index != static_cast<int>(index)| to make sure
+        // we check if |start_index == static_cast<int>(index)| to make sure
         // that we don't get stuck in a loop if the packet buffer is filled
         // with packets of the same timestamp.
-        if (is_h264 && start_index != static_cast<int>(index) &&
+        if ((is_h264 && start_index == static_cast<int>(index)) ||
             (!sequence_buffer_[start_index].used ||
              data_buffer_[start_index].timestamp != frame_timestamp)) {
           break;