Add a 'filter' option to zip_build to allow for build post-processing.

Add a 'filter' option to zip_build to allow for build post-processing.

ASAN is the first filter. On windows, it will substitute in the .asan version
in place of the original exe when building the zip file.

R=nsylvain


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=170495

[iannucci, 2012-11-28 03:20:57]

Cut out of https://chromiumcodereview.appspot.com/11379003/

[nsylvain, 2012-11-28 18:55:52]

LGTM, although I still think that we should just overwrite the files instead of
creating new ones.

https://chromiumcodereview.appspot.com/11413207/diff/1/scripts/common/chromiu...
File scripts/common/chromium_utils.py (right):

https://chromiumcodereview.appspot.com/11413207/diff/1/scripts/common/chromiu...
scripts/common/chromium_utils.py:537: if new_src_path is None:
Why do you break if it's None?  Maybe another replacement function can do
something about it?  

Does it really need to be a list of replacement functions? 1 function is not
enough?

[iannucci, 2012-11-29 02:19:12]

https://chromiumcodereview.appspot.com/11413207/diff/1/scripts/common/chromiu...
File scripts/common/chromium_utils.py (right):

https://chromiumcodereview.appspot.com/11413207/diff/1/scripts/common/chromiu...
scripts/common/chromium_utils.py:537: if new_src_path is None:
On 2012/11/28 18:55:52, nsylvain wrote:
> Why do you break if it's None?  Maybe another replacement function can do
> something about it?  

The next function would get None as it's argument... not sure what it could
intelligently do about it.

> 
> Does it really need to be a list of replacement functions? 1 function is not
> enough? 

Yeah this is a fair point. I was thinking about this the other day, and now that
you brought it up, I'll fix it for sure. Check the new patch.

[nsylvain, 2012-11-30 01:55:38]

lgtm

[commit-bot: I haz the power, 2012-11-30 18:37:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/iannucci@chromium.org/11413207/1003

[commit-bot: I haz the power, 2012-11-30 18:42:37]

Change committed as 170495