A few small cleanups of stuff caught by lint

A few small cleanups of stuff caught by lint

Committed: https://crrev.com/264087f45a32c67c888869865bc331bb894e7b49
Cr-Commit-Position: refs/heads/master@{#12412}

[ossu, 2016-04-08 12:33:08]

ossu@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[ossu, 2016-04-08 12:33:23]

Put the cleanups here instead. :)

[kwiberg-webrtc, 2016-04-09 06:41:08]

lgtm

Consider changing the commit message to e.g. "A few small cleanups of stuff
caught by lint" (note: no period at the end).

[ossu, 2016-04-11 08:15:19]

Sure. Any special reason for it other than, I guess, consistency?

[kwiberg-webrtc, 2016-04-11 08:17:20]

Description was changed from

==========
Did a few small cleanups of stuff caught by lint.
==========

to

==========
A few small cleanups of stuff caught by lint
==========

[kwiberg-webrtc, 2016-04-11 08:23:41]

On 2016/04/11 08:15:19, ossu wrote:
> Sure. Any special reason for it other than, I guess, consistency?

The style I suggested removes words and punctuation that don't contribute
anything. That the committer did the change is obvious from the context...

[kwiberg-webrtc, 2016-04-11 08:25:21]

On 2016/04/11 08:23:41, kwiberg-webrtc wrote:
> On 2016/04/11 08:15:19, ossu wrote:
> > Sure. Any special reason for it other than, I guess, consistency?
> 
> The style I suggested removes words and punctuation that don't contribute
> anything. That the committer did the change is obvious from the context...

(Also, note that you have to change both the subject and the first line of the
description (I did it for you). I believe the former is used for e.g. the e-mail
subject line, while the latter is what'll end up in the revision control
system.)

[ossu, 2016-04-12 10:35:39]

The CQ bit was checked by ossu@webrtc.org

[commit-bot: I haz the power, 2016-04-12 10:35:48]

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

[commit-bot: I haz the power, 2016-04-12 12:36:20]

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

[commit-bot: I haz the power, 2016-04-12 12:36:21]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[ossu, 2016-04-18 13:45:21]

The CQ bit was checked by ossu@webrtc.org

[commit-bot: I haz the power, 2016-04-18 13:45:31]

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

[commit-bot: I haz the power, 2016-04-18 15:07:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-18 15:07:38]

Description was changed from

==========
A few small cleanups of stuff caught by lint
==========

to

==========
A few small cleanups of stuff caught by lint

Committed: https://crrev.com/264087f45a32c67c888869865bc331bb894e7b49
Cr-Commit-Position: refs/heads/master@{#12412}
==========

[commit-bot: I haz the power, 2016-04-18 15:07:39]

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