New issue
Advanced search Search tips

Issue 764064 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 757232



Sign in to add a comment

Replace std::deque with base::circular_deque in QUIC code

Project Member Reported by brettw@chromium.org, Sep 11 2017

Issue description

This needs a shim since this code is shared with some other non-Chromium projects.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e9713155cc1998cf2baace2dd56949fe03cba610

commit e9713155cc1998cf2baace2dd56949fe03cba610
Author: Ryan Hamilton <rch@chromium.org>
Date: Tue Sep 12 23:09:51 2017

Add new QuicQueue and QuicDeque classes to wrap std::queue and std::deque internally, and base::queue/base::deque in chromium.

Merge internal change: 168283215

Bug: 764064
Change-Id: Ic1b74c515866ab8ba57eeff55c3f1137dd6ee031
Reviewed-on: https://chromium-review.googlesource.com/661790
Reviewed-by: Jana Iyengar <jri@chromium.org>
Reviewed-by: Brett Wilson <brettw@chromium.org>
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501442}
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/crypto/quic_crypto_client_config.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/crypto/quic_crypto_client_config.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/frames/quic_ack_frame.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/frames/quic_ack_frame.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/packet_number_indexed_queue.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/quic_connection.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/quic_header_list.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/quic_headers_stream.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/quic_stream_send_buffer.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/quic_stream_send_buffer.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/core/quic_unacked_packet_map.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/platform/api/quic_containers.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/platform/impl/quic_containers_impl.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/test_tools/simulator/link.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/test_tools/simulator/link.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/test_tools/simulator/queue.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/test_tools/simulator/queue.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/quic/test_tools/simulator/switch.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/tools/quic/quic_simple_server_session.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/tools/quic/quic_simple_server_session_test.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/tools/quic/quic_time_wait_list_manager.h
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/tools/quic/test_tools/server_thread.cc
[modify] https://crrev.com/e9713155cc1998cf2baace2dd56949fe03cba610/net/tools/quic/test_tools/server_thread.h

Sign in to add a comment