Address comments in 8921029 extract_from_cab script now takes a global lock while extracting fro...

Address comments in 8921029  extract_from_cab script now takes a global lock while extracting from the CAB file.

http://codereview.chromium.org/8921029/

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

[M-A Ruel, 2011-12-13 02:16:26]

http://codereview.chromium.org/8929001/diff/4/build/extract_from_cab.py
File build/extract_from_cab.py (right):

http://codereview.chromium.org/8929001/diff/4/build/extract_from_cab.py#newco...
build/extract_from_cab.py:25: time.sleep(10)
in fact, make it 100ms.

http://codereview.chromium.org/8929001/diff/4/build/extract_from_cab.py#newco...
build/extract_from_cab.py:45: if level != 0:
return on line 43 instead.

[apatrick_chromium, 2011-12-13 02:20:58]

http://codereview.chromium.org/8929001/diff/4/build/extract_from_cab.py
File build/extract_from_cab.py (right):

http://codereview.chromium.org/8929001/diff/4/build/extract_from_cab.py#newco...
build/extract_from_cab.py:25: time.sleep(10)
On 2011/12/13 02:16:26, Marc-Antoine Ruel wrote:
> in fact, make it 100ms.

Done.

http://codereview.chromium.org/8929001/diff/4/build/extract_from_cab.py#newco...
build/extract_from_cab.py:45: if level != 0:
On 2011/12/13 02:16:26, Marc-Antoine Ruel wrote:
> return on line 43 instead.

If it succeeds it is intended to touch the extracted file on #54.

[M-A Ruel, 2011-12-13 02:22:12]

lgtm

[commit-bot: I haz the power, 2011-12-13 02:27:32]

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

[commit-bot: I haz the power, 2011-12-13 03:32:18]

Change committed as 114173