Auto-roll script: Add dirty tree check and git pull

Auto-roll script: Add dirty tree check and git pull

BUG=4688
R=pbos@webrtc.org

Committed: https://crrev.com/1b76ca121ed820495b3ae238e06df0728b42ece9
Cr-Commit-Position: refs/heads/master@{#9399}

[kjellander_webrtc, 2015-06-09 09:26:34]

kjellander@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2015-06-09 09:34:32]

https://codereview.webrtc.org/1173523003/diff/1/tools/autoroller/roll_chromiu...
File tools/autoroller/roll_chromium_revision.py (right):

https://codereview.webrtc.org/1173523003/diff/1/tools/autoroller/roll_chromiu...
tools/autoroller/roll_chromium_revision.py:302: _RunCommand(['git', 'pull'])
Do you want to do this under a dry run? Also is there any sanity check possible
that you've not applied commits onto master?

[kjellander_webrtc, 2015-06-09 10:51:04]

I added a dirty tree check and updated according to your comment.

https://codereview.webrtc.org/1173523003/diff/1/tools/autoroller/roll_chromiu...
File tools/autoroller/roll_chromium_revision.py (right):

https://codereview.webrtc.org/1173523003/diff/1/tools/autoroller/roll_chromiu...
tools/autoroller/roll_chromium_revision.py:302: _RunCommand(['git', 'pull'])
On 2015/06/09 09:34:31, pbos-webrtc wrote:
> Do you want to do this under a dry run? 

I thought it shouldn't never hurt your checkout to run it, but sure, let's not
do it.

> Also is there any sanity check possible that you've not applied commits onto
master?

I added a check for a dirty tree, but I think it's hard to check that no local
commits have been made on master, let's skip that.

[pbos-webrtc, 2015-06-09 10:57:26]

lgtm

[kjellander_webrtc, 2015-06-09 10:59:00]

Committed patchset #2 (id:20001) manually as
1b76ca121ed820495b3ae238e06df0728b42ece9 (presubmit successful).

[commit-bot: I haz the power, 2015-06-09 10:59:08]

Patchset 2 (id:??) landed as
https://crrev.com/1b76ca121ed820495b3ae238e06df0728b42ece9
Cr-Commit-Position: refs/heads/master@{#9399}