telemetry: remove dependency on FormBasedCredentialsBackend as its deprecated. |
||||||||||||||
Issue descriptionIn running power_LoadTest on M-53 noticed this warning ... 11:58:37 WARNI| /usr/local/telemetry/src/third_party/catapult/telemetry/telemetry/internal/browser/browser_credentials.py:28: UserWarning: Class FormBasedCredentialsBackend is deprecated. It will no longer be supported on May 05, 2017. Please remove it or switch to an alternative before that time. Use the login helper modules in https://code.google.com/p/chromium/codesearch#chromium/src/tools/perf/page_sets/login_helpers/ instead. codepen_credentials_backend.CodePenCredentialsBackend(),
,
Jun 3 2016
I see this on all chromeos test runs. I was assuming this was something transient on all platforms, but I guess not.
,
Jun 3 2016
Albert, in case there's someone on the team that you'd like to load balance this to. It should be pretty easy to track down. Otherwise I'll take a look next week.
,
Jun 8 2016
Greg, Albert suggested I assign this to you to take a look. The easiest way to reproduce this is to run the following command: <path_to_chromium_src>/src/third_party/catapult/telemetry/bin/run_tests --browser=cros-chrome --remote=<DUT_IP> testBrowserCreation You should see messages about various CredentialsBackends being deprecated. Please investigate.
,
Jul 14 2016
Looks like this is no longer happening. Will verify when I'm back in Mtn View.
,
Jul 14 2016
Agreed ... no longer see it in latest M-53 or M-54. For example, https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/69508687-chromeos-test/chromeos4-row8-rack3-host15/power_LoadTest
,
Jul 14 2016
Tests that use form_base-credentials_backend will still trigger this warning. (see https://codereview.chromium.org/2050863003/)
,
Jul 14 2016
Ah .. yes perhaps too hasty a comment ... I should simply state that power_LoadTest runs aren't showing it and leave status unchanged then. Thanks for clarifying
,
Oct 25 2016
Todd: Is this still happening?
,
Oct 25 2016
Not seeing it on power_LoadTest at least.
,
Oct 25 2016
Thanks!
,
Jan 21 2017
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by nedngu...@google.com
, May 25 2016