[mb] Add presubmit checks

[mb] Add presubmit checks

Also comment out some currently unused configs the presubmit
check now complains about. They'll be used soon, once
respective bots are added.

BUG=chromium:616035
TBR=kjellander@chromium.org
NOTRY=true

Committed: https://crrev.com/3249e23e5a8b2eb56a1178d1b128defce465fcc5
Cr-Commit-Position: refs/heads/master@{#37056}

[Michael Achenbach, 2016-06-17 08:21:15]

Description was changed from

==========
[mb] Add presubmit checks

BUG=
==========

to

==========
[mb] Add presubmit checks

Also comment out some currently unused configs the presubmit
check now complains about. They'll be used soon, once
respective bots are added.

BUG=chromium:616035
TBR=kjellander@chromium.org
NOTRY=true
==========

[Michael Achenbach, 2016-06-17 08:21:15]

machenbach@chromium.org changed reviewers:
+ kjellander@chromium.org

[Michael Achenbach, 2016-06-17 08:23:07]

machenbach@chromium.org changed reviewers:
+ jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-06-17 08:23:08]

PTAL

https://codereview.chromium.org/2076053002/diff/1/infra/mb/PRESUBMIT.py
File infra/mb/PRESUBMIT.py (right):

https://codereview.chromium.org/2076053002/diff/1/infra/mb/PRESUBMIT.py#newcode1
infra/mb/PRESUBMIT.py:1: # Copyright 2016 the V8 project authors. All rights
reserved.
Shamelessly copied from webrtc. Thanks for the hint Henrik!

Usually we keep the original copyright header and add one line about v8. Not
sure if really needed within the chromium community.

[kjellander_chromium, 2016-06-17 08:25:21]

lgtm

https://codereview.chromium.org/2076053002/diff/1/infra/mb/PRESUBMIT.py
File infra/mb/PRESUBMIT.py (right):

https://codereview.chromium.org/2076053002/diff/1/infra/mb/PRESUBMIT.py#newcode1
infra/mb/PRESUBMIT.py:1: # Copyright 2016 the V8 project authors. All rights
reserved.
On 2016/06/17 08:23:07, Michael Achenbach wrote:
> Shamelessly copied from webrtc. Thanks for the hint Henrik!
> 
> Usually we keep the original copyright header and add one line about v8. Not
> sure if really needed within the chromium community.

Happy to help! It has caught several errors I've made already.

[Michael Achenbach, 2016-06-17 08:26:35]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-06-17 08:26:48]

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

[commit-bot: I haz the power, 2016-06-17 08:34:51]

Description was changed from

==========
[mb] Add presubmit checks

Also comment out some currently unused configs the presubmit
check now complains about. They'll be used soon, once
respective bots are added.

BUG=chromium:616035
TBR=kjellander@chromium.org
NOTRY=true
==========

to

==========
[mb] Add presubmit checks

Also comment out some currently unused configs the presubmit
check now complains about. They'll be used soon, once
respective bots are added.

BUG=chromium:616035
TBR=kjellander@chromium.org
NOTRY=true
==========

[commit-bot: I haz the power, 2016-06-17 08:34:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-17 08:35:22]

Description was changed from

==========
[mb] Add presubmit checks

Also comment out some currently unused configs the presubmit
check now complains about. They'll be used soon, once
respective bots are added.

BUG=chromium:616035
TBR=kjellander@chromium.org
NOTRY=true
==========

to

==========
[mb] Add presubmit checks

Also comment out some currently unused configs the presubmit
check now complains about. They'll be used soon, once
respective bots are added.

BUG=chromium:616035
TBR=kjellander@chromium.org
NOTRY=true

Committed: https://crrev.com/3249e23e5a8b2eb56a1178d1b128defce465fcc5
Cr-Commit-Position: refs/heads/master@{#37056}
==========

[commit-bot: I haz the power, 2016-06-17 08:35:22]

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