Style fix and docstring for the APM QA Python tool

This CL fixes the following:
- snake_case -> CapWords
- compulsory docstring added
- style

A followup CL will fix remaining issues as raised by the next version of the WebRTC Python linter (update in progress).

BUG=webrtc:7218
NOTRY=True

Review-Url: https://codereview.webrtc.org/2793903006
Cr-Commit-Position: refs/heads/master@{#17543}
Committed: https://chromium.googlesource.com/external/webrtc/+/dea682d25b60820af862cfb8f35a761ef1597784

[AleBzk, 2017-04-05 12:54:49]

Description was changed from

==========
snake_case -> CapWords

BUG=webrtc:7218
NOTRY=True
==========

to

==========
This CL fixes the following:
- snake_case -> CapWords
- compulsory docstring added
- style

A followup CL will fix remaining issues as raised by the next version of the
WebRTC Python linter (update in progress).

BUG=webrtc:7218
NOTRY=True
==========

[AleBzk, 2017-04-05 12:54:50]

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

[AleBzk, 2017-04-05 13:07:07]

As agreed in https://codereview.webrtc.org/2718063003/, I finally fixed the
style of the APM QA tool.

[kjellander_webrtc, 2017-04-05 14:13:48]

lgtm but consider my comment.

https://codereview.webrtc.org/2793903006/diff/20001/webrtc/modules/audio_proc...
File
webrtc/modules/audio_processing/test/py_quality_assessment/apm_quality_assessment_export.py
(right):

https://codereview.webrtc.org/2793903006/diff/20001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/test/py_quality_assessment/apm_quality_assessment_export.py:33:
def InstanceArgumentsParser():
The style guide encourages _ for internal names. Why not keep it?
https://google.github.io/styleguide/pyguide.html?showone=Naming#Naming

[AleBzk, 2017-04-05 14:22:56]

The CQ bit was checked by alessiob@webrtc.org

[AleBzk, 2017-04-05 14:22:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
kjellander@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2793903006/#ps40001 (title:
"_ prefix for internal use functions")

[commit-bot: I haz the power, 2017-04-05 14:23:00]

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, 2017-04-05 14:25:35]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491402176962720,
"parent_rev": "129fc9ce9402c445184edcc1053b1864d9428771", "commit_rev":
"dea682d25b60820af862cfb8f35a761ef1597784"}

[commit-bot: I haz the power, 2017-04-05 14:25:39]

Description was changed from

==========
This CL fixes the following:
- snake_case -> CapWords
- compulsory docstring added
- style

A followup CL will fix remaining issues as raised by the next version of the
WebRTC Python linter (update in progress).

BUG=webrtc:7218
NOTRY=True
==========

to

==========
This CL fixes the following:
- snake_case -> CapWords
- compulsory docstring added
- style

A followup CL will fix remaining issues as raised by the next version of the
WebRTC Python linter (update in progress).

BUG=webrtc:7218
NOTRY=True

Review-Url: https://codereview.webrtc.org/2793903006
Cr-Commit-Position: refs/heads/master@{#17543}
Committed:
https://chromium.googlesource.com/external/webrtc/+/dea682d25b60820af862cfb8f...
==========

[commit-bot: I haz the power, 2017-04-05 14:25:41]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/dea682d25b60820af862cfb8f...

[AleBzk, 2017-04-05 14:28:51]

https://codereview.webrtc.org/2793903006/diff/20001/webrtc/modules/audio_proc...
File
webrtc/modules/audio_processing/test/py_quality_assessment/apm_quality_assessment_export.py
(right):

https://codereview.webrtc.org/2793903006/diff/20001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/test/py_quality_assessment/apm_quality_assessment_export.py:33:
def InstanceArgumentsParser():
On 2017/04/05 14:13:48, kjellander_webrtc wrote:
> The style guide encourages _ for internal names. Why not keep it?
> https://google.github.io/styleguide/pyguide.html?showone=Naming#Naming

Done.