login_utils_browsertest.cc is flaky.

login_utils_browsertest.cc is flaky.

BUG=158839
TEST=tests pass.
TBR=nkostylev@chromium.org

[commit-bot: I haz the power, 2012-10-31 23:54:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/achuith@chromium.org/11358028/1

[commit-bot: I haz the power, 2012-11-01 04:32:10]

List of reviewers changed. joaodasilva@chromium.org,nkostylev@chromium.org did a
drive-by without LGTM'ing!

[commit-bot: I haz the power, 2012-11-01 04:51:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/achuith@chromium.org/11358028/1

[commit-bot: I haz the power, 2012-11-01 04:51:36]

Change committed as 165294

[Nikita (slow), 2012-11-01 18:54:45]

lgtm but I think you should have just disabled
LoginUtilsBlockingLoginTest.EnterpriseLoginBlocksForEnterpriseUser

[achuithb, 2012-11-01 19:10:20]

On 2012/11/01 18:54:45, Nikita Kostylev wrote:
> lgtm but I think you should have just disabled
> LoginUtilsBlockingLoginTest.EnterpriseLoginBlocksForEnterpriseUser

I've seen NormalLoginDoesntBlock fail as well, though not as often.

[achuithb, 2012-11-01 19:11:17]

On 2012/11/01 19:10:20, achuith.bhandarkar wrote:
> On 2012/11/01 18:54:45, Nikita Kostylev wrote:
> > lgtm but I think you should have just disabled
> > LoginUtilsBlockingLoginTest.EnterpriseLoginBlocksForEnterpriseUser
> 
> I've seen NormalLoginDoesntBlock fail as well, though not as often.

Here's an example:
http://build.chromium.org/p/tryserver.chromium/builders/linux_chromeos/builds...

[Nikita (slow), 2012-11-01 19:22:20]

makes sense to disable whole test then, thanks!

lgtm