Implements a Zip() utility function. Refactor existing

Implements a Zip() utility function. Refactor existing
Unzip-relatedness into shared locations.

[Aaron Boodman, 2009-05-29 22:58:27]

Sorry about the lack of correct formatting of copies in the patch. I tried for
awhile again today to get git-cl to do this correctly and again failed.

[Erik does not do reviews, 2009-05-29 23:33:27]

LGTM - just add a TODO for the other test

http://codereview.chromium.org/118028/diff/1024/1032
File chrome/common/zip.h (right):

http://codereview.chromium.org/118028/diff/1024/1032#newcode13
Line 13: // directory. If it is a directory, it will be zipped recursively.
note that on failure a partial zip file may still exist and needs to be cleaned
up by the caller.

http://codereview.chromium.org/118028/diff/1024/1033
File chrome/common/zip_unittest.cc (right):

http://codereview.chromium.org/118028/diff/1024/1033#newcode124
Line 124: ASSERT_TRUE(Zip(src_dir, zip_file));
we should do a zip then unzip verification test

http://codereview.chromium.org/118028/diff/1024/1047
File tools/memory_watcher/memory_watcher.vcproj (right):

http://codereview.chromium.org/118028/diff/1024/1047#newcode2
Line 2: <VisualStudioProject Keyword="Win32Proj" Name="memory_watcher"
ProjectGUID="{3BD81303-4E14-4559-AA69-B30C3BAB08DD}" ProjectType="Visual C++"
RootNamespace="memory_watcher" Version="8.00">
this diff seems unintentional