Default download_from_google_storage.py to verbose

Default download_from_google_storage.py to verbose

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

[hinoka, 2015-07-06 18:16:53]

hinoka@chromium.org changed reviewers:
+ friedman@chromium.org, iannucci@chromium.org

[hinoka, 2015-07-06 18:16:54]

mostly harmless flag change CL, ptal.

[friedman1, 2015-07-06 18:31:03]

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

[friedman1, 2015-07-06 18:31:04]

https://codereview.chromium.org/1223793008/diff/20001/download_from_google_st...
File download_from_google_storage.py (right):

https://codereview.chromium.org/1223793008/diff/20001/download_from_google_st...
download_from_google_storage.py:364: parser.add_option('--no-verbose',
action='store_false', dest='verbose',
-q, --quiet  instead? and then remove the deprecated verbose options?

[friedman1, 2015-07-06 18:31:18]

On 2015/07/06 18:31:04, friedman1 wrote:
>
https://codereview.chromium.org/1223793008/diff/20001/download_from_google_st...
> File download_from_google_storage.py (right):
> 
>
https://codereview.chromium.org/1223793008/diff/20001/download_from_google_st...
> download_from_google_storage.py:364: parser.add_option('--no-verbose',
> action='store_false', dest='verbose',
> -q, --quiet  instead? and then remove the deprecated verbose options?

Thanks for taking this on!!

[iannucci, 2015-07-06 18:35:23]

lgtm (though --quiet would be more normal :))

[hinoka, 2015-09-25 18:25:05]

The CQ bit was checked by hinoka@chromium.org

[hinoka, 2015-09-25 18:25:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/1223793008/#ps40001
(title: "--quet")

[commit-bot: I haz the power, 2015-09-25 18:25:14]

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

[commit-bot: I haz the power, 2015-09-25 18:27:45]

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

[commit-bot: I haz the power, 2015-09-25 18:27:46]

Try jobs failed on following builders:
  depot_tools_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/depot_tools_pre...)

[hinoka, 2015-09-25 18:28:28]

The CQ bit was checked by hinoka@chromium.org

[hinoka, 2015-09-25 18:28:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/1223793008/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-09-25 18:28:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1223793008/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1223793008/60001

[commit-bot: I haz the power, 2015-09-25 18:33:42]

Committed patchset #4 (id:60001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=296882