Zlib compressor for Chromoting

remoting/base/compressor_zlib_unittest.cc

Index: remoting/base/compressor_zlib_unittest.cc
diff --git a/remoting/base/compressor_zlib_unittest.cc b/remoting/base/compressor_zlib_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..9d65aad404819fda96cc6b64ce38946bf43cfc9e
--- /dev/null
+++ b/remoting/base/compressor_zlib_unittest.cc
@@ -0,0 +1,52 @@
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include <stdlib.h>
+
+#include "remoting/base/compressor_zlib.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+static void GenerateTestData(uint8* data, int size, int seed) {
+  srand(seed);
+  for (int i = 0; i < size; ++i)
+    data[i] = rand() % 256;
+}
+
+// Keep compressing |input_data| into |output_data| until the last
+// bytes is consumed.

[dmac, 2010/07/01 00:12:54]

s/bytes/byte/

+static void Compress(remoting::Compressor* compressor,
+                     const uint8* input_data, int input_size,
+                     uint8* output_data, int output_size) {
+
+  // Feed data into the compress until the end.
+  // This loop will rewrite |output_data| continuously.
+  while (input_size) {
+    int consumed, written;
+    compressor->Write(input_data, input_size,
+                      output_data, output_size,
+                      &consumed, &written);
+    input_data += consumed;
+    input_size -= consumed;
+  }
+
+  // And then flush the remaining data from the compressor.
+  int written;
+  while (compressor->Flush(output_data, output_size, &written)) {
+  }
+}
+
+TEST(CompressorZlibTest, SimpleCompress) {
+  static const int kRawDataSize = 1024 * 128;
+  static const int kCompressedDataSize = 256;
+  uint8 raw_data[kRawDataSize];
+  uint8 compressed_data[kCompressedDataSize];
+
+  // Generate the test data.g
+  GenerateTestData(raw_data, kRawDataSize, 99);
+
+  // Then use the compressor to compress.
+  remoting::CompressorZlib compressor;
+  Compress(&compressor, raw_data, kRawDataSize,

[dmac, 2010/07/01 00:12:54]

there's actually no test here other than it doesn't crash. Do you want to try
testing it with a known data set as well to verify that you get the expected
values out? This may be overkill. Up to you.

[Alpha Left Google, 2010/07/01 00:14:57]

The test will become invalid if zlib is updated and the compressed format
doesn't match.

I plan instead to do a compress + decompress test when I write the decompressor.

+           compressed_data, kCompressedDataSize);
+}