Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(503)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 14863012: Remove bzip2 code since it is no longer being used. (Closed)

Created:
7 years, 7 months ago by Lei Zhang
Modified:
7 years, 7 months ago
CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Remove bzip2 code since it is no longer being used. BUG=237998 TBR=tommi Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=200116

Patch Set 1 #

Patch Set 2 : add missing file #

Patch Set 3 : add another missing file #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5637 lines) Patch
M build/all.gyp View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M build/common.gypi View 1 2 1 chunk +0 lines, -1 line 0 comments Download
D build/linux/unbundle/bzip2.gyp View 1 2 1 chunk +0 lines, -22 lines 0 comments Download
M build/linux/unbundle/replace_gyp_files.py View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_browser.gypi View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_browser_chromeos.gypi View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_browser_extensions.gypi View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_browser_ui.gypi View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_common.gypi View 1 2 2 chunks +0 lines, -2 lines 0 comments Download
M chrome/chrome_tests_unit.gypi View 1 2 3 chunks +0 lines, -3 lines 0 comments Download
D chrome/common/bzip2_error_handler.cc View 1 2 1 chunk +0 lines, -14 lines 0 comments Download
D chrome/common/bzip2_unittest.cc View 1 2 1 chunk +0 lines, -72 lines 0 comments Download
M chrome_frame/chrome_frame.gyp View 1 2 1 chunk +0 lines, -1 line 0 comments Download
D third_party/bzip2/LICENSE View 1 2 1 chunk +0 lines, -42 lines 0 comments Download
D third_party/bzip2/README View 1 2 1 chunk +0 lines, -215 lines 0 comments Download
D third_party/bzip2/README.chromium View 1 2 1 chunk +0 lines, -12 lines 0 comments Download
D third_party/bzip2/blocksort.c View 1 2 1 chunk +0 lines, -1094 lines 0 comments Download
D third_party/bzip2/bzip2.gyp View 1 2 1 chunk +0 lines, -79 lines 0 comments Download
D third_party/bzip2/bzlib.h View 1 2 1 chunk +0 lines, -282 lines 0 comments Download
D third_party/bzip2/bzlib.c View 1 2 1 chunk +0 lines, -1572 lines 0 comments Download
D third_party/bzip2/bzlib_private.h View 1 2 1 chunk +0 lines, -509 lines 0 comments Download
D third_party/bzip2/compress.c View 1 2 1 chunk +0 lines, -672 lines 0 comments Download
D third_party/bzip2/crctable.c View 1 2 1 chunk +0 lines, -104 lines 0 comments Download
D third_party/bzip2/decompress.c View 1 2 1 chunk +0 lines, -646 lines 0 comments Download
D third_party/bzip2/huffman.c View 1 2 1 chunk +0 lines, -205 lines 0 comments Download
D third_party/bzip2/randtable.c View 1 2 1 chunk +0 lines, -84 lines 0 comments Download

Messages

Total messages: 17 (0 generated)
Lei Zhang
7 years, 7 months ago (2013-05-07 21:54:58 UTC) #1
Lei Zhang
7 years, 7 months ago (2013-05-09 19:48:31 UTC) #2
Paweł Hajdan Jr.
LGTM
7 years, 7 months ago (2013-05-09 21:23:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/14863012/1
7 years, 7 months ago (2013-05-09 21:59:40 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/14863012/15001
7 years, 7 months ago (2013-05-09 22:17:20 UTC) #5
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=2269
7 years, 7 months ago (2013-05-09 22:37:59 UTC) #6
Lei Zhang
cpu: I need an OWNERS review for third_party.
7 years, 7 months ago (2013-05-09 23:01:06 UTC) #7
cpu_(ooo_6.6-7.5)
lgtm Note that you don't need owners review for modifications, only to add new third ...
7 years, 7 months ago (2013-05-10 17:35:22 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/14863012/15001
7 years, 7 months ago (2013-05-10 17:36:27 UTC) #9
commit-bot: I haz the power
Retried try job too often on win_x64_rel for step(s) compile http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_rel&number=12137
7 years, 7 months ago (2013-05-10 17:56:11 UTC) #10
Lei Zhang
On 2013/05/10 17:56:11, I haz the power (commit-bot) wrote: > Retried try job too often ...
7 years, 7 months ago (2013-05-10 23:43:45 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/14863012/46001
7 years, 7 months ago (2013-05-10 23:44:06 UTC) #12
commit-bot: I haz the power
Retried try job too often on linux_rel for step(s) interactive_ui_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&number=126644
7 years, 7 months ago (2013-05-11 00:42:56 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/14863012/46001
7 years, 7 months ago (2013-05-13 22:14:26 UTC) #14
commit-bot: I haz the power
Retried try job too often on android_dbg for step(s) slave_steps http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_dbg&number=62870
7 years, 7 months ago (2013-05-13 23:24:18 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/14863012/46001
7 years, 7 months ago (2013-05-14 23:22:04 UTC) #16
commit-bot: I haz the power
7 years, 7 months ago (2013-05-15 00:13:58 UTC) #17
Message was sent while issue was closed. 
Change committed as 200116

Powered by Google App Engine
This is Rietveld 408576698