Rip out bzip compression for native sources.

Rip out bzip compression for native sources.


R=vogelheim@chromium.org

Committed: https://chromium.googlesource.com/v8/v8/+/63722c2f9960b0b0347f67288ed6907b4570f0be

[vogelheim1, 2014-11-21 11:50:50]

vogelheim@google.com changed reviewers:
+ vogelheim@google.com

[vogelheim1, 2014-11-21 11:50:50]

lgtm - provided caveats below are solved.

I think this may be missing two cases:
- BUILD.gn
- cctest/cctest.gyp

[vogelheim1, 2014-11-21 11:51:10]

On 2014/11/21 11:50:50, vogelheim1 wrote:
> lgtm - provided caveats below are solved.
> 
> I think this may be missing two cases:
> - BUILD.gn
> - cctest/cctest.gyp

Oh wait, cctest.gyp was in. But BUILD.gn is missing. :)

[Yang, 2014-11-21 11:54:05]

On 2014/11/21 11:51:10, vogelheim1 wrote:
> On 2014/11/21 11:50:50, vogelheim1 wrote:
> > lgtm - provided caveats below are solved.
> > 
> > I think this may be missing two cases:
> > - BUILD.gn
> > - cctest/cctest.gyp
> 
> Oh wait, cctest.gyp was in. But BUILD.gn is missing. :)

ah yes. Forgot the gn build as usual :/

[Yang, 2014-11-21 11:54:09]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2014-11-21 11:54:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/750543002/20001

[vogelheim1, 2014-11-21 12:02:04]

On 2014/11/21 11:54:05, Yang wrote:
> On 2014/11/21 11:51:10, vogelheim1 wrote:
> > On 2014/11/21 11:50:50, vogelheim1 wrote:
> > > lgtm - provided caveats below are solved.
> > > 
> > > I think this may be missing two cases:
> > > - BUILD.gn
> > > - cctest/cctest.gyp
> > 
> > Oh wait, cctest.gyp was in. But BUILD.gn is missing. :)
> 
> ah yes. Forgot the gn build as usual :/

One more thing: This is an API change, so a note should end up in our API change
doc.

[commit-bot: I haz the power, 2014-11-21 12:07:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-21 12:07:08]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[Yang, 2014-11-21 12:16:24]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2014-11-21 12:17:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/750543002/40001

[commit-bot: I haz the power, 2014-11-21 12:45:16]

Committed patchset #3 (id:40001)

[jb.seo, 2016-10-12 08:12:08]

lifeofthanks@gmail.com changed reviewers:
+ lifeofthanks@gmail.com

[jb.seo, 2016-10-12 08:12:09]

Hello, vogelheim.

It's old thread. But,
can I ask why this happens?

I think the compression gives benefit in some cases.
Is there any reason to rip out the compression related codes?