Remember what branches we delete and do not try to reparent them in cleanup.

Remember what branches we delete and do not try to reparent them in cleanup.

R=agable@chromium.org
BUG=357367
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=260704

[agable, 2014-03-29 00:02:04]

LGTM

https://codereview.chromium.org/214133006/diff/1/tests/git_rebase_update_test.py
File tests/git_rebase_update_test.py (right):

https://codereview.chromium.org/214133006/diff/1/tests/git_rebase_update_test...
tests/git_rebase_update_test.py:133: self.assertIn('Deleted branch
empty_branch2', output)
See today's ToT: Would be better to have many smaller tests of rebase-update
that each test one feature of it (deleting empty branches, reparenting children
of deleted branches, etc.)

[iannucci, 2014-03-31 23:33:47]

On 2014/03/29 00:02:04, agable wrote:
> LGTM
> 
>
https://codereview.chromium.org/214133006/diff/1/tests/git_rebase_update_test.py
> File tests/git_rebase_update_test.py (right):
> 
>
https://codereview.chromium.org/214133006/diff/1/tests/git_rebase_update_test...
> tests/git_rebase_update_test.py:133: self.assertIn('Deleted branch
> empty_branch2', output)
> See today's ToT: Would be better to have many smaller tests of rebase-update
> that each test one feature of it (deleting empty branches, reparenting
children
> of deleted branches, etc.)

Yes, nice things would be good to have... We should refactor the tests, but I'm
not planning to do it in this CL.

[iannucci, 2014-03-31 23:34:06]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2014-03-31 23:34:16]

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

[commit-bot: I haz the power, 2014-03-31 23:36:11]

Change committed as 260704