chromite: check response status code

chromite: check response status code

ChromiteFetcher._GetText does not check response status code. Presumably
it caused
https://luci-milo.appspot.com/swarming/prod/2f976557c60ffb10/steps/gclient_runhooks/0/stdout

R=dnj@chromium.org
BUG=622950
Committed: https://chromium.googlesource.com/chromium/tools/build/+/bad355222fd53b0e8e238b7c84dab67f23e6f93c

[nodir, 2016-06-29 01:50:25]

PTAL

[dnj (Google), 2016-06-29 01:57:41]

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

[dnj (Google), 2016-06-29 01:57:41]

lgtm

[nodir, 2016-06-29 01:59:54]

The CQ bit was checked by nodir@chromium.org

[commit-bot: I haz the power, 2016-06-29 02:00:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 02:03:23]

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

[commit-bot: I haz the power, 2016-06-29 02:03:23]

Try jobs failed on following builders:
  Build Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[nodir, 2016-06-29 15:28:48]

The CQ bit was checked by nodir@chromium.org

[nodir, 2016-06-29 15:28:48]

The patchset sent to the CQ was uploaded after l-g-t-m from dnj@google.com
Link to the patchset: https://codereview.chromium.org/2108713004/#ps20001
(title: "fix request")

[commit-bot: I haz the power, 2016-06-29 15:28:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 15:33:05]

Description was changed from

==========
chromite: check response status code

ChromiteFetcher._GetText does not check response status code. Presumably
it caused
https://luci-milo.appspot.com/swarming/prod/2f976557c60ffb10/steps/gclient_ru...

R=dnj@chromium.org
BUG=622950
==========

to

==========
chromite: check response status code

ChromiteFetcher._GetText does not check response status code. Presumably
it caused
https://luci-milo.appspot.com/swarming/prod/2f976557c60ffb10/steps/gclient_ru...

R=dnj@chromium.org
BUG=622950

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/bad355222fd53b0e8e23...
==========

[commit-bot: I haz the power, 2016-06-29 15:33:06]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/tools/build/+/bad355222fd53b0e8e23...

[commit-bot: I haz the power, 2016-06-29 15:33:11]

CQ bit was unchecked.