Clean up comments and docstrings in webkitpy.common.system.logtesting.

Clean up comments and docstrings in webkitpy.common.system.logtesting.

This CL is a minor clean-up CL, and only touches comments and whitespace.

In this CL:
 - Make formatting consistent with other modules
 - Remove FIXME note about renaming LoggingTestCase to LoggingTestCaseBase,
   since in practice LoggingTestCase is not interpreted as a test case
   since the logtesting module doesn't end in _unittest, and even if it
   were, LoggingTestCase has no test methods.

Committed: https://crrev.com/832443eec1ac13b31d044967e8e36469bc4d28c5
Cr-Commit-Position: refs/heads/master@{#421331}

[qyearsley, 2016-09-27 17:13:08]

Description was changed from

==========
Clean up comments and docstrings in webkitpy.common.system.logtesting.

In particular:
 - Make formatting consistent with other modules
 - Remove FIXME note about renaming LoggingTestCase to LoggingTestCaseBase,
   since in practice LoggingTestCase is not interpreted as a test case
   since the logtesting module doesn't end in _unittest, and even if it
   were, LoggingTestCase has no test methods.
==========

to

==========
Clean up comments and docstrings in webkitpy.common.system.logtesting.

This CL is a minor clean-up CL, and only touches comments and whitespace.

In this CL:
 - Make formatting consistent with other modules
 - Remove FIXME note about renaming LoggingTestCase to LoggingTestCaseBase,
   since in practice LoggingTestCase is not interpreted as a test case
   since the logtesting module doesn't end in _unittest, and even if it
   were, LoggingTestCase has no test methods.
==========

[qyearsley, 2016-09-27 17:13:32]

qyearsley@chromium.org changed reviewers:
+ jeffcarp@chromium.org

[qyearsley, 2016-09-27 17:17:04]

The CQ bit was checked by qyearsley@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-27 17:17:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 20:29:29]

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

[commit-bot: I haz the power, 2016-09-27 20:29:29]

Dry run: This issue passed the CQ dry run.

[jeffcarp, 2016-09-27 20:48:31]

On 2016/09/27 at 17:15:37, qyearsley wrote:
> 

lgtm :D

[qyearsley, 2016-09-27 20:49:20]

The CQ bit was checked by qyearsley@chromium.org

[jeffcarp, 2016-09-27 20:49:27]

On 2016/09/27 at 20:48:31, jeffcarp wrote:
> On 2016/09/27 at 17:15:37, qyearsley wrote:
> > 
> 
> lgtm :D

oops, did this not require an lgtm?

[commit-bot: I haz the power, 2016-09-27 20:49:40]

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

[jeffcarp, 2016-09-27 20:51:57]

On 2016/09/27 at 20:49:40, commit-bot wrote:
> CQ is trying da patch. Follow status at
>  
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

Ah, I think I mistook the CQ dry run as the actual run.

[commit-bot: I haz the power, 2016-09-27 20:56:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 21:01:20]

Description was changed from

==========
Clean up comments and docstrings in webkitpy.common.system.logtesting.

This CL is a minor clean-up CL, and only touches comments and whitespace.

In this CL:
 - Make formatting consistent with other modules
 - Remove FIXME note about renaming LoggingTestCase to LoggingTestCaseBase,
   since in practice LoggingTestCase is not interpreted as a test case
   since the logtesting module doesn't end in _unittest, and even if it
   were, LoggingTestCase has no test methods.
==========

to

==========
Clean up comments and docstrings in webkitpy.common.system.logtesting.

This CL is a minor clean-up CL, and only touches comments and whitespace.

In this CL:
 - Make formatting consistent with other modules
 - Remove FIXME note about renaming LoggingTestCase to LoggingTestCaseBase,
   since in practice LoggingTestCase is not interpreted as a test case
   since the logtesting module doesn't end in _unittest, and even if it
   were, LoggingTestCase has no test methods.

Committed: https://crrev.com/832443eec1ac13b31d044967e8e36469bc4d28c5
Cr-Commit-Position: refs/heads/master@{#421331}
==========

[commit-bot: I haz the power, 2016-09-27 21:01:22]

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