Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

NOTRY=True

Committed: https://crrev.com/df9e4d9fce7c0477299bdafb22aad59715f591e3
Cr-Commit-Position: refs/heads/master@{#13677}

[aleloi, 2016-08-08 14:21:34]

I think the new mixer module folder should be added to CPPLINT_DIRS.

https://codereview.webrtc.org/2228503003/diff/1/webrtc/modules/audio_mixer/so...
File webrtc/modules/audio_mixer/source/new_audio_conference_mixer_impl.h
(right):

https://codereview.webrtc.org/2228503003/diff/1/webrtc/modules/audio_mixer/so...
webrtc/modules/audio_mixer/source/new_audio_conference_mixer_impl.h:17: #include
<vector>
git cl presubmit checks found missing include.

[aleloi, 2016-08-08 14:22:07]

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

[aleloi, 2016-08-08 14:22:31]

I think the new mixer module folder should be added to CPPLINT_DIRS.

[kjellander_webrtc, 2016-08-08 14:49:04]

Nice to see new paths being added to lint checking.

Make sure to set NOTRY=True if you're going to land this with the CQ.

[kjellander_webrtc, 2016-08-08 14:49:17]

... and lgtm

[aleloi, 2016-08-08 14:49:36]

Description was changed from

==========
Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.
==========

to

==========
Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

NOTRY=True
==========

[aleloi, 2016-08-08 14:49:43]

The CQ bit was checked by aleloi@webrtc.org

[commit-bot: I haz the power, 2016-08-08 14:49:54]

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

[commit-bot: I haz the power, 2016-08-08 14:49:55]

This CL has an open dependency (Issue 2221443002 Patch 100001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[aleloi, 2016-08-08 14:50:08]

The CQ bit was unchecked by aleloi@webrtc.org

[aleloi2, 2016-08-08 17:23:18]

The CQ bit was checked by aleloi@google.com

[commit-bot: I haz the power, 2016-08-08 17:23:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

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

Description was changed from

==========
Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

NOTRY=True
==========

to

==========
Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

NOTRY=True
==========

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

Committed patchset #1 (id:1)

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

Description was changed from

==========
Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

NOTRY=True
==========

to

==========
Added new mixer folder to CPPLINT_DIRS. Fixed one small issue.

NOTRY=True

Committed: https://crrev.com/df9e4d9fce7c0477299bdafb22aad59715f591e3
Cr-Commit-Position: refs/heads/master@{#13677}
==========

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

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