Chromium Code Reviews
DescriptionRevert of [Telemetry] Add temporary logging to desktop_browser_finder (patchset #1 id:1 of https://codereview.chromium.org/2567783002/ )
Reason for revert:
No longer needed; root cause diagnosed.
Original issue's description:
> [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
TBR=sullivan@chromium.org,nednguyen@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=catapult:#3074
Review-Url: https://codereview.chromium.org/2574483002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/60a406396e81bc09500e854916c51657972ad033
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|
|||||||||||||||||||