[Telemetry] Add temporary logging to desktop_browser_finder

[Telemetry] Add temporary logging to desktop_browser_finder

We've seen scenarios where DesktopBrowserFinder unit tests fail because
sys.platform reports a wrong platform, presumably due to being stubbed out too
aggressively by system_stub. This CL adds some temporary logging to make sure
that the reason browsers sometimes can't be found on the Catapult CQ is due to
this stubbing problem.

BUG=catapult:#3074
NOTRY=true

Review-Url: https://codereview.chromium.org/2567783002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/707aaac64b3c0a86d253e1ab502e73996d63927e

[eakuefner, 2016-12-09 20:07:00]

eakuefner@chromium.org changed reviewers:
+ nednguyen@google.com, sullivan@chromium.org

[eakuefner, 2016-12-09 20:07:01]

PTAL. NOTRY because I don't want to deal with the flake.

[nednguyen, 2016-12-09 20:08:23]

lgtm

[eakuefner, 2016-12-09 20:08:28]

The CQ bit was checked by eakuefner@chromium.org

[commit-bot: I haz the power, 2016-12-09 20:08:36]

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-12-09 20:10:12]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481314108775520, "parent_rev":
"df2363501bd70a2c99ac8a0307a1b864cedf7db6", "commit_rev":
"707aaac64b3c0a86d253e1ab502e73996d63927e"}

[commit-bot: I haz the power, 2016-12-09 20:10:14]

Description was changed from

==========
[Telemetry] Add temporary logging to desktop_browser_finder

We've seen scenarios where DesktopBrowserFinder unit tests fail because
sys.platform reports a wrong platform, presumably due to being stubbed out too
aggressively by system_stub. This CL adds some temporary logging to make sure
that the reason browsers sometimes can't be found on the Catapult CQ is due to
this stubbing problem.

BUG=catapult:#3074
NOTRY=true
==========

to

==========
[Telemetry] Add temporary logging to desktop_browser_finder

We've seen scenarios where DesktopBrowserFinder unit tests fail because
sys.platform reports a wrong platform, presumably due to being stubbed out too
aggressively by system_stub. This CL adds some temporary logging to make sure
that the reason browsers sometimes can't be found on the Catapult CQ is due to
this stubbing problem.

BUG=catapult:#3074
NOTRY=true

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

[commit-bot: I haz the power, 2016-12-09 20:10:15]

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

[eakuefner, 2016-12-12 19:13:04]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2574483002/ by eakuefner@chromium.org.

The reason for reverting is: No longer needed; root cause diagnosed..