New issue
Advanced search Search tips

Issue 597145 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 597791



Sign in to add a comment

Add compression to Blimp streams.

Project Member Reported by kmarshall@chromium.org, Mar 23 2016

Issue description

Add some kind of compression to Blimp streams.
 
Blocking: 597791
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 30 2016

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

commit 38ad3ca2674316ec799b678a69fe3a7bfd5f0edf
Author: kmarshall <kmarshall@chromium.org>
Date: Wed Mar 30 18:30:11 2016

Blimp: add packet-level DEFLATE compression using zlib.

Compresses the packets in the stream as length-prefixed DEFLATE blocks.
Blocks are transmitted in a single, long-lived stream with a
compression window that is retained across consecutive compression
calls.
* Create new classes: CompressedPacketReader and CompressedPacketWriter.
* Integrate CompressedPacketReader/CompressedPacketWriter into application.
* Make PacketReader resize its input GrowableIOBuffer as needed.
* Add third_party/zlib DEP.

R=wez@chromium.org,agl@chromium.org
BUG= 597145 

Review URL: https://codereview.chromium.org/1825263003

Cr-Commit-Position: refs/heads/master@{#384029}

[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/BUILD.gn
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/DEPS
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_reader.cc
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_reader.h
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_unittest.cc
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_writer.cc
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_writer.h
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/stream_packet_reader.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/stream_packet_reader_unittest.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/stream_socket_connection.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/test_common.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/test_common.h

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 30 2016

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

commit 38ad3ca2674316ec799b678a69fe3a7bfd5f0edf
Author: kmarshall <kmarshall@chromium.org>
Date: Wed Mar 30 18:30:11 2016

Blimp: add packet-level DEFLATE compression using zlib.

Compresses the packets in the stream as length-prefixed DEFLATE blocks.
Blocks are transmitted in a single, long-lived stream with a
compression window that is retained across consecutive compression
calls.
* Create new classes: CompressedPacketReader and CompressedPacketWriter.
* Integrate CompressedPacketReader/CompressedPacketWriter into application.
* Make PacketReader resize its input GrowableIOBuffer as needed.
* Add third_party/zlib DEP.

R=wez@chromium.org,agl@chromium.org
BUG= 597145 

Review URL: https://codereview.chromium.org/1825263003

Cr-Commit-Position: refs/heads/master@{#384029}

[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/BUILD.gn
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/DEPS
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_reader.cc
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_reader.h
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_unittest.cc
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_writer.cc
[add] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/compressed_packet_writer.h
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/stream_packet_reader.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/stream_packet_reader_unittest.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/stream_socket_connection.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/test_common.cc
[modify] https://crrev.com/38ad3ca2674316ec799b678a69fe3a7bfd5f0edf/blimp/net/test_common.h

Status: Fixed (was: Assigned)
Labels: Archive-Blimp

Sign in to add a comment