Revert of Don't check bucket permissions if --no-auth.

Revert of Don't check bucket permissions if --no-auth. (patchset #1 id:1 of https://codereview.chromium.org/772203002/)

Reason for revert:
Seems to break download_from_google_storage.py in gclient runhooks:

Traceback (most recent call last):
  File "/b/depot_tools/download_from_google_storage.py", line 469, in <module>
    sys.exit(main(sys.argv))
  File "/b/depot_tools/download_from_google_storage.py", line 458, in main
    code = check_bucket_permissions(base_url, gsutil, options.no_auth)
TypeError: check_bucket_permissions() takes exactly 2 arguments (3 given)

Original issue's description:
> Don't check bucket permissions if --no-auth.
> 
> Checking bucket permissions takes ~400ms. We don't
> need to do this if --no-auth because we know we
> won't get a 403 and the 404 check will be handled
> later when we try to actually download the file.
> 
> Also, remove the check for a null bucket. This
> can't happen since we will throw a parser error
> in the main function before we get to this code.
> 
> R=iannucci@chromium.org
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=293250

TBR=agable@chromium.org,iannucci@chromium.org,ojan@chromium.org
NOTREECHECKS=true
NOTRY=true

[Vadim Sh., 2014-12-04 22:37:52]

Created Revert of Don't check bucket permissions if --no-auth.

[commit-bot: I haz the power, 2014-12-04 22:38:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/784433003/1

[Vadim Sh., 2014-12-04 22:38:49]

One example:
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...

I'm sure there are lot more...

[agable, 2014-12-04 22:39:04]

On 2014/12/04 at 22:38:49, vadimsh wrote:
> One example:
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...
> 
> I'm sure there are lot more...

LGTM, found this at the same time as you.

[commit-bot: I haz the power, 2014-12-04 22:39:44]

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

[commit-bot: I haz the power, 2014-12-04 22:39:45]

Presubmit check for 784433003-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 Messages **
--tbr was specified, skipping OWNERS check

** Presubmit ERRORS **
Pylint (110 files) (61.85s) failed
/b/depot_tools/third_party/logilab/astroid/modutils.py:520: UserWarning: Module
six was already imported from /b/depot_tools/third_party/six/__init__.py, but
/usr/local/lib/python2.7/dist-packages is being added to sys.path
  import pkg_resources
Problem importing module .svn: No module named .svn
Problem importing module .svn: No module named .svn
************* Module fix_encoding
E:232, 0: Slice index is not an int, None, or instance with __index__
(invalid-slice-index)
************* Module gclient_scm
E:301,18: No value for argument 'print_func' in constructor call
(no-value-for-parameter)
E:1191,18: No value for argument 'print_func' in constructor call
(no-value-for-parameter)


post-dcommit-hook-test.sh failed
Command
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests/post-dcommit-hook-test.sh
returned non-zero exit status 1 in
/b/infra_internal/commit_queue/workdir/tools/depot_tools/tests
Setting up test SVN repo...
Setting up test git-svn repo...
TESTING: dcommitted code
Using 50% similarity for rename/copy detection. Override with --similarity.
Description:
dcommit
TESTING: post-cl-dcommit hook executed
FAILURE: post-cl-dcommit hook executed


Presubmit checks took 97.8s to calculate.

[agable, 2014-12-04 22:40:19]

On 2014/12/04 at 22:39:04, agable wrote:
> On 2014/12/04 at 22:38:49, vadimsh wrote:
> > One example:
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...
> > 
> > I'm sure there are lot more...
> 
> LGTM, found this at the same time as you.

Also, dcommit this, since the depot_tools cq doesn't work.

[Vadim Sh., 2014-12-04 22:45:42]

Ojan landed the fix instead.

[ojan, 2014-12-04 22:46:01]

Fixed in https://codereview.chromium.org/780113003.

Sorry for breaking the world. :(