Refactor ZLib and Verbatim encoders.

remoting/remoting.gyp

 # 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.
 
 {
   'variables': {
     'chromium_code': 1,
   },
 
   'target_defaults': {
@@ skipping 117 matching lines @@
       ],
       # This target needs a hard dependency because dependent targets
       # depend on chromotocol_proto_lib for headers.
       'hard_dependency': 1,
       'sources': [
         'base/capture_data.cc',
         'base/capture_data.h',
         'base/compound_buffer.cc',
         'base/compound_buffer.h',
         'base/compressor.h',
+        'base/compressor_verbatim.cc',
+        'base/compressor_verbatim.h',
         'base/compressor_zlib.cc',
         'base/compressor_zlib.h',
         'base/constants.cc',
         'base/constants.h',
         'base/decoder.h',
         'base/decoder_vp8.cc',
         'base/decoder_vp8.h',
         'base/decoder_row_based.cc',
         'base/decoder_row_based.h',
         'base/decompressor.h',
         'base/decompressor_verbatim.cc',
         'base/decompressor_verbatim.h',
         'base/decompressor_zlib.cc',
         'base/decompressor_zlib.h',
         'base/encoder.h',
-        'base/encoder_verbatim.cc',
-        'base/encoder_verbatim.h',
         'base/encoder_vp8.cc',
         'base/encoder_vp8.h',
-        'base/encoder_zlib.cc',
-        'base/encoder_zlib.h',
+        'base/encoder_row_based.cc',
+        'base/encoder_row_based.h',
         'base/tracer.cc',
         'base/tracer.h',
         'base/types.h',
         'base/util.cc',
         'base/util.h',
       ],
       'conditions': [
         ['target_arch=="arm"', {
           'sources!': [
             'base/decoder_vp8.cc',
@@ skipping 281 matching lines @@
         '../testing/gmock/include',
       ],
       'sources': [
 # BUG57351        'base/codec_test.cc',
 # BUG57351        'base/codec_test.h',
         'base/compound_buffer_unittest.cc',
         'base/compressor_zlib_unittest.cc',
 # BUG57351        'base/decoder_vp8_unittest.cc',
         'base/decompressor_zlib_unittest.cc',
 # BUG57351        'base/encode_decode_unittest.cc',
-# BUG57351        'base/encoder_verbatim_unittest.cc',
 # BUG57351        'base/encoder_vp8_unittest.cc',
-# BUG57351        'base/encoder_zlib_unittest.cc',
+# BUG57351        'base/encoder_row_based_unittest.cc',

[Alpha Left Google, 2010/12/02 19:39:55]

enable this test case?

[Sergey Ulanov, 2010/12/03 00:17:21]

This test requires codec_test.cc, and the code in codec_test.cc is very stale. I
will fix it in a separate CL.

         'base/mock_objects.h',
 # BUG57351        'client/chromoting_view_unittest.cc',
         'client/mock_objects.h',
         'host/access_verifier_unittest.cc',
         'host/chromoting_host_context_unittest.cc',
         'host/differ_unittest.cc',
         'host/differ_block_unittest.cc',
         'host/heartbeat_sender_unittest.cc',
         'host/host_key_pair_unittest.cc',
         'host/json_host_config_unittest.cc',
@@ skipping 52 matching lines @@
       ],  # end of 'conditions'
     },  # end of target 'chromoting_unittests'
   ],  # end of targets
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: