Fix upload.py to properly use a local logger

Fix upload.py to properly use a local logger

When rietveld.py overrides upload.py's logging module, it generally works fine...
until someone tries to run upload.py with any level of verbosity. Then the
calls to logging.getLogger and logging.INFO/logging.DEBUG fail. This patches
upload.py to properly use a module-scope logger so that rietveld.py doesn't
have to perform invasive surgery, and everything just works.

This version of upload.py taken from upstream Rietveld at:
changeset:   1267:d7b39eca7dbe
branch:      chromium

R=djacques@chromium.org, maruel@chromium.org

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

[M-A Ruel, 2014-04-02 01:29:03]

upload.py changes need to go upstream first.

[agable, 2014-04-02 01:35:27]

On 2014/04/02 01:29:03, M-A Ruel wrote:
> upload.py changes need to go upstream first.

https://codereview.appspot.com/83380043/

[agable, 2014-04-05 00:27:18]

On 2014/04/02 01:35:27, agable wrote:
> On 2014/04/02 01:29:03, M-A Ruel wrote:
> > upload.py changes need to go upstream first.
> 
> https://codereview.appspot.com/83380043/

Rietveld CL has landed, so ping here.

[M-A Ruel, 2014-04-05 01:23:26]

On 2014/04/05 00:27:18, agable wrote:
> On 2014/04/02 01:35:27, agable wrote:
> > On 2014/04/02 01:29:03, M-A Ruel wrote:
> > > upload.py changes need to go upstream first.
> > 
> > https://codereview.appspot.com/83380043/
> 
> Rietveld CL has landed, so ping here.

Update the readme first please

[agable, 2014-04-05 01:57:15]

On 2014/04/05 01:23:26, M-A Ruel wrote:
> On 2014/04/05 00:27:18, agable wrote:
> > On 2014/04/02 01:35:27, agable wrote:
> > > On 2014/04/02 01:29:03, M-A Ruel wrote:
> > > > upload.py changes need to go upstream first.
> > > 
> > > https://codereview.appspot.com/83380043/
> > 
> > Rietveld CL has landed, so ping here.
> 
> Update the readme first please

What readme? None of the readme's in depot_tools contain diffs, and there isn't
a readme in depot_tools/third_party.

[agable, 2014-04-09 19:04:10]

On 2014/04/05 01:57:15, agable wrote:
> On 2014/04/05 01:23:26, M-A Ruel wrote:
> > On 2014/04/05 00:27:18, agable wrote:
> > > On 2014/04/02 01:35:27, agable wrote:
> > > > On 2014/04/02 01:29:03, M-A Ruel wrote:
> > > > > upload.py changes need to go upstream first.
> > > > 
> > > > https://codereview.appspot.com/83380043/
> > > 
> > > Rietveld CL has landed, so ping here.
> > 
> > Update the readme first please
> 
> What readme? None of the readme's in depot_tools contain diffs, and there
isn't
> a readme in depot_tools/third_party.

Ping

[M-A Ruel, 2014-04-09 20:04:40]

On 2014/04/09 19:04:10, agable wrote:
> On 2014/04/05 01:57:15, agable wrote:
> > On 2014/04/05 01:23:26, M-A Ruel wrote:
> > > On 2014/04/05 00:27:18, agable wrote:
> > > > On 2014/04/02 01:35:27, agable wrote:
> > > > > On 2014/04/02 01:29:03, M-A Ruel wrote:
> > > > > > upload.py changes need to go upstream first.
> > > > > 
> > > > > https://codereview.appspot.com/83380043/
> > > > 
> > > > Rietveld CL has landed, so ping here.
> > > 
> > > Update the readme first please
> > 
> > What readme? None of the readme's in depot_tools contain diffs, and there
> isn't
> > a readme in depot_tools/third_party.
> 
> Ping

Argh, right. Can you note the revision you took upload.py from in the commit
message?
lgtm

[agable, 2014-04-09 20:17:06]

On 2014/04/09 20:04:40, M-A Ruel wrote:
> On 2014/04/09 19:04:10, agable wrote:
> > On 2014/04/05 01:57:15, agable wrote:
> > > On 2014/04/05 01:23:26, M-A Ruel wrote:
> > > > On 2014/04/05 00:27:18, agable wrote:
> > > > > On 2014/04/02 01:35:27, agable wrote:
> > > > > > On 2014/04/02 01:29:03, M-A Ruel wrote:
> > > > > > > upload.py changes need to go upstream first.
> > > > > > 
> > > > > > https://codereview.appspot.com/83380043/
> > > > > 
> > > > > Rietveld CL has landed, so ping here.
> > > > 
> > > > Update the readme first please
> > > 
> > > What readme? None of the readme's in depot_tools contain diffs, and there
> > isn't
> > > a readme in depot_tools/third_party.
> > 
> > Ping
> 
> Argh, right. Can you note the revision you took upload.py from in the commit
> message?
> lgtm

Done.

[agable, 2014-04-09 20:17:15]

The CQ bit was checked by agable@chromium.org

[commit-bot: I haz the power, 2014-04-09 20:17:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/agable@chromium.org/221423007/1

[commit-bot: I haz the power, 2014-04-09 20:19:32]

Change committed as 262793