Revert of [devil] Extract logging common behavior to its own module.

Revert of [devil] Extract logging common behavior to its own module. (patchset #3 id:40001 of https://codereview.chromium.org/2972253002/ )

Reason for revert:
Appears to be blocking catapult roller. Tests are failing with, e.g.:

Traceback (most recent call last):
  File "/b/swarming/w/ir/build/android/test_runner.py", line 34, in <module>
    from devil.utils import run_tests_helper
  File "/b/swarming/w/ir/third_party/catapult/devil/devil/utils/run_tests_helper.py", line 10, in <module>
    from devil.utils import logging_common
ImportError: cannot import name logging_common
[E2017-08-10T18:17:37.717840Z 31228 0 subcommand_run.go:333] Command completed with non-zero return code. {"returnCode":1}
[I2017-08-10T18:17:37.718115Z 31228 0 main.go:349] Terminating.                                {"returnCode":1}

https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/240094

Original issue's description:
> [devil] Extract logging common behavior to its own module.
> 
> Also deprecates the now-poorly-named run_tests_helper.py.
> 
> Review-Url: https://codereview.chromium.org/2972253002
> Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/18709dd6bd88aee18994e1570754e0a1bb92bea4

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2998833002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/f9ea41b51d8f6125b8283a2a2c4b610db431149c

[perezju, 2017-08-11 08:00:20]

The CQ bit was checked by [email protected]

[perezju, 2017-08-11 08:00:20]

Created Revert of [devil] Extract logging common behavior to its own module.

[commit-bot: I haz the power, 2017-08-11 08:00:24]

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, 2017-08-11 08:00:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1502438420326750, "parent_rev":
"1b0009b023c32818960159f122f218a45bd472a4", "commit_rev":
"f9ea41b51d8f6125b8283a2a2c4b610db431149c"}

[commit-bot: I haz the power, 2017-08-11 08:00:35]

Description was changed from

==========
Revert of [devil] Extract logging common behavior to its own module. (patchset
#3 id:40001 of https://codereview.chromium.org/2972253002/ )

Reason for revert:
Appears to be blocking catapult roller. Tests are failing with, e.g.:

Traceback (most recent call last):
  File "/b/swarming/w/ir/build/android/test_runner.py", line 34, in <module>
    from devil.utils import run_tests_helper
  File
"/b/swarming/w/ir/third_party/catapult/devil/devil/utils/run_tests_helper.py",
line 10, in <module>
    from devil.utils import logging_common
ImportError: cannot import name logging_common
[E2017-08-10T18:17:37.717840Z 31228 0 subcommand_run.go:333] Command completed
with non-zero return code. {"returnCode":1}
[I2017-08-10T18:17:37.718115Z 31228 0 main.go:349] Terminating.                 
              {"returnCode":1}

https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...

Original issue's description:
> [devil] Extract logging common behavior to its own module.
> 
> Also deprecates the now-poorly-named run_tests_helper.py.
> 
> Review-Url: https://codereview.chromium.org/2972253002
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of [devil] Extract logging common behavior to its own module. (patchset
#3 id:40001 of https://codereview.chromium.org/2972253002/ )

Reason for revert:
Appears to be blocking catapult roller. Tests are failing with, e.g.:

Traceback (most recent call last):
  File "/b/swarming/w/ir/build/android/test_runner.py", line 34, in <module>
    from devil.utils import run_tests_helper
  File
"/b/swarming/w/ir/third_party/catapult/devil/devil/utils/run_tests_helper.py",
line 10, in <module>
    from devil.utils import logging_common
ImportError: cannot import name logging_common
[E2017-08-10T18:17:37.717840Z 31228 0 subcommand_run.go:333] Command completed
with non-zero return code. {"returnCode":1}
[I2017-08-10T18:17:37.718115Z 31228 0 main.go:349] Terminating.                 
              {"returnCode":1}

https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...

Original issue's description:
> [devil] Extract logging common behavior to its own module.
> 
> Also deprecates the now-poorly-named run_tests_helper.py.
> 
> Review-Url: https://codereview.chromium.org/2972253002
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2998833002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-08-11 08:00:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[jbudorick, 2017-08-11 17:39:52]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/3000813002/ by [email protected].

The reason for reverting is: Needs a src/-side change. Will manually handle the
roll..