Speculative fix for build on windows build bots.

Speculative fix for build on windows build bots.

The root of problem is a _cache_temp file.
git_cache expected that it is a folder.
So rmtree failed to remove it.

BUG=
TBR= dpranke@chromium.org, enne@chromium.org
NOTRY=true

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

[loislo, 2014-12-25 10:58:46]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-12-25 10:58:55]

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

[commit-bot: I haz the power, 2014-12-25 11:01:46]

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

[commit-bot: I haz the power, 2014-12-25 11:01:47]

Presubmit check for 825133002-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 (113 files) (124.12s) failed
************* Module android
W: 12, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)
************* Module blink
W: 13, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)
************* Module ios
W: 12, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)


tests/git_common_test.py (6.01s) failed
...F..................u.......
======================================================================
FAIL: testGetBranchesInfo (__main__.GitMutableFunctionsTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/git_common_test.py", line 418, in testGetBranchesInfo
    self.assertEquals(expected, actual)
AssertionError: {'': None, 'happybranch': ('abcee4d', 'master', 1, None),
'parent_gone': ('d8774 [truncated]... != {'': None, 'happybranch':
BranchesInfo(hash='abcee4d', upstream='master', ahead= [truncated]...
  {'': None,
-  'child': ('abcee4d', 'happybranch', None, None),
-  'happybranch': ('abcee4d', 'master', 1, None),
-  'master': ('d87747e', '', None, None),
-  'parent_gone': ('d87747e', 'to_delete', 1, None),
+  'child': BranchesInfo(hash='abcee4d', upstream='happybranch', ahead=None,
behind=None),
+  'happybranch': BranchesInfo(hash='abcee4d', upstream='master', ahead=1,
behind=None),
+  'master': BranchesInfo(hash='d87747e', upstream='', ahead=None, behind=None),
+  'parent_gone': BranchesInfo(hash='d87747e', upstream='to_delete',
ahead=36373792, behind=170),
   'to_delete': None}

----------------------------------------------------------------------
Ran 30 tests in 5.791s

FAILED (failures=1, unexpected successes=1)
Name         Stmts   Miss  Cover   Missing
------------------------------------------
git_common     373      0   100%   


Presubmit checks took 167.2s to calculate.

Was the presubmit check useful? If not, run "git cl presubmit -v"
to figure out which PRESUBMIT.py was run, then run git blame
on the file to figure out who to ask for help.

[loislo, 2014-12-25 13:05:18]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-12-25 13:05:29]

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

[commit-bot: I haz the power, 2014-12-25 13:08:13]

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

[commit-bot: I haz the power, 2014-12-25 13:08:14]

Presubmit check for 825133002-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 (113 files) (121.36s) failed
************* Module android
W: 12, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)
************* Module blink
W: 13, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)
************* Module ios
W: 12, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)


tests/git_common_test.py (5.61s) failed
...F..................u.......
======================================================================
FAIL: testGetBranchesInfo (__main__.GitMutableFunctionsTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/git_common_test.py", line 418, in testGetBranchesInfo
    self.assertEquals(expected, actual)
AssertionError: {'': None, 'happybranch': ('39431ea', 'master', 1, None),
'parent_gone': ('7defc [truncated]... != {'': None, 'happybranch':
BranchesInfo(hash='39431ea', upstream='master', ahead= [truncated]...
  {'': None,
-  'child': ('39431ea', 'happybranch', None, None),
-  'happybranch': ('39431ea', 'master', 1, None),
-  'master': ('7defc06', '', None, None),
-  'parent_gone': ('7defc06', 'to_delete', 1, None),
+  'child': BranchesInfo(hash='39431ea', upstream='happybranch', ahead=None,
behind=None),
+  'happybranch': BranchesInfo(hash='39431ea', upstream='master', ahead=1,
behind=None),
+  'master': BranchesInfo(hash='7defc06', upstream='', ahead=None, behind=None),
+  'parent_gone': BranchesInfo(hash='7defc06', upstream='to_delete',
ahead=32953632, behind=170),
   'to_delete': None}

----------------------------------------------------------------------
Ran 30 tests in 5.382s

FAILED (failures=1, unexpected successes=1)
Name         Stmts   Miss  Cover   Missing
------------------------------------------
git_common     373      0   100%   


Presubmit checks took 161.0s to calculate.

[loislo, 2014-12-25 13:22:27]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-12-25 13:22:45]

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

[alph, 2014-12-25 13:23:16]

alph@chromium.org changed reviewers:
+ alph@chromium.org

[commit-bot: I haz the power, 2014-12-25 13:25:31]

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

[commit-bot: I haz the power, 2014-12-25 13:25:32]

Presubmit check for 825133002-20001 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 (113 files) (123.86s) failed
************* Module android
W: 12, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)
************* Module blink
W: 13, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)
************* Module ios
W: 12, 0: Method 'expected_root' is abstract in class 'Recipe' but is not
overridden (abstract-method)


Presubmit checks took 162.1s to calculate.

[loislo, 2014-12-25 13:44:42]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-12-25 13:45:44]

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

[commit-bot: I haz the power, 2014-12-25 13:48:39]

Committed patchset #3 (id:40001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=293502