Print gsutil error message when we enconter as 403 or 404

Print gsutil error message when we enconter as 403 or 404
We appear to be getting 403's when encountering errors other than mismatched
credentials.  The error message was originally hidden to avoid clutter, but it
has become obvious that there are other transient errors hidden behind a 403.

This change prints the original gsutil error message regardless of whether or not
we enconter a 403 or 404.


BUG=

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

[Isaac (away), 2013-09-19 17:57:59]

lgtm, thanks!

https://codereview.chromium.org/23484061/diff/1/download_from_google_storage.py
File download_from_google_storage.py (right):

https://codereview.chromium.org/23484061/diff/1/download_from_google_storage....
download_from_google_storage.py:90: print >> sys.stderr, ls_err
Optional nit: consider wrapping this so it's clear where the text is coming
from, perhaps:

'gsutil: "%s"' % ls_err

Though gsutil already does a pretty good just of informative output, for
example:
GSResponseError: status=403, code=RequestTimeTooSkewed, reason=Forbidden

[Ryan Tseng, 2013-09-19 18:05:50]

https://codereview.chromium.org/23484061/diff/1/download_from_google_storage.py
File download_from_google_storage.py (right):

https://codereview.chromium.org/23484061/diff/1/download_from_google_storage....
download_from_google_storage.py:90: print >> sys.stderr, ls_err
On 2013/09/19 17:57:59, Isaac wrote:
> Optional nit: consider wrapping this so it's clear where the text is coming
> from, perhaps:
> 
> 'gsutil: "%s"' % ls_err
> 
> Though gsutil already does a pretty good just of informative output, for
> example:
> GSResponseError: status=403, code=RequestTimeTooSkewed, reason=Forbidden

Yeah I think gsutil already does a pretty good job, no need to add any more
colons

$ download_from_google_storage . -b chsihd-dhef -dr                             
                                                                                
      
GSResponseError: status=404, code=NoSuchBucket, reason=Not Found.

gs://chsihd-dhef not found.

vs.

mudkip(er) ~/Documents/depot_tools$ download_from_google_storage . -b
chsihd-dhef -dr
gsutil: GSResponseError: status=404, code=NoSuchBucket, reason=Not Found.

gs://chsihd-dhef not found.

[commit-bot: I haz the power, 2013-09-19 18:06:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hinoka@google.com/23484061/1

[commit-bot: I haz the power, 2013-09-19 18:09:44]

Presubmit check for 23484061-1 failed and returned exit status 1.

Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    depot_tools/download_from_google_storage.py

Presubmit checks took 179.6s to calculate.

[Ryan Tseng, 2013-09-19 18:15:08]

+iannucci: Can I get an OWNERS stamp from you? :)

[iannucci, 2013-09-19 18:59:55]

lgtm

[commit-bot: I haz the power, 2013-09-19 19:58:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hinoka@google.com/23484061/1

[commit-bot: I haz the power, 2013-09-19 20:00:53]

Change committed as 224195