Make cros login error message more descriptive.

Make cros login error message more descriptive.

When a telemetry test fails to login the test user, the most likely cause
is a failure to load the login extension. It is difficult to identify this
problem with the old generic timeout error message.

BUG=239212
TEST=run_tests with bad --auth-ext-path flag on cros
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=201928

[Tim Song, 2013-05-23 22:23:34]

This is a CL on the telemetry side when the auth extension fails to load. PTAL.

[achuithb, 2013-05-23 22:36:55]

I think this is ok. Nat?

https://codereview.chromium.org/15747016/diff/1/tools/telemetry/telemetry/cor...
File tools/telemetry/telemetry/core/chrome/cros_util.py (right):

https://codereview.chromium.org/15747016/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/chrome/cros_util.py:84: 'extension passed through
--auth-ext-path is valid and has belongs '
remove 'has'

[Tim Song, 2013-05-23 23:02:33]

https://codereview.chromium.org/15747016/diff/1/tools/telemetry/telemetry/cor...
File tools/telemetry/telemetry/core/chrome/cros_util.py (right):

https://codereview.chromium.org/15747016/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/chrome/cros_util.py:84: 'extension passed through
--auth-ext-path is valid and has belongs '
On 2013/05/23 22:36:55, achuith.bhandarkar wrote:
> remove 'has'

Done.

[nduca, 2013-05-23 23:17:48]

lgtm

[achuithb, 2013-05-23 23:18:21]

lgtm

[commit-bot: I haz the power, 2013-05-23 23:20:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tengs@chromium.org/15747016/3

[commit-bot: I haz the power, 2013-05-23 23:36:02]

Change committed as 201928