Enable TestLoadingPublicUsersFromLocalState

Enable TestLoadingPublicUsersFromLocalState

Ran this test 100 times locally with no failures. Didn't fail on bots either

BUG=393708
Committed: https://crrev.com/f189d5356a18e7b5e159c8028184137eb8656e06
Cr-Commit-Position: refs/heads/master@{#347898}

[achuithb, 2015-09-08 11:21:11]

achuith@chromium.org changed reviewers:
+ bartfab@chromium.org

[achuithb, 2015-09-08 11:21:11]

Bartosz, please take a look.

I've run this test locally 100+ times, and on the bots a few times. 

Without a stack (not in the bug), and expired links to failed runs, no further
investigation is possible. I'd like to re-enable the test.

[bartfab (slow), 2015-09-08 14:57:17]

https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
File chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
(right):

https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
chrome/browser/chromeos/login/existing_user_controller_browsertest.cc:463:
runner2 = new content::MessageLoopRunner;
Oops, that was quite a stupid typo. Thanks for fixing the test!

[achuithb, 2015-09-09 08:50:31]

On 2015/09/08 14:57:17, bartfab wrote:
>
https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
> File chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
> (right):
> 
>
https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
> chrome/browser/chromeos/login/existing_user_controller_browsertest.cc:463:
> runner2 = new content::MessageLoopRunner;
> Oops, that was quite a stupid typo. Thanks for fixing the test!

Are you still looking at this, or did you forget to lgtm?

[achuithb, 2015-09-09 08:50:38]

The CQ bit was checked by achuith@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-09-09 08:51:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1319373003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1319373003/1

[bartfab (slow), 2015-09-09 09:04:58]

On 2015/09/09 08:50:31, achuithb wrote:
> On 2015/09/08 14:57:17, bartfab wrote:
> >
>
https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
> > File chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
> > (right):
> > 
> >
>
https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
> > chrome/browser/chromeos/login/existing_user_controller_browsertest.cc:463:
> > runner2 = new content::MessageLoopRunner;
> > Oops, that was quite a stupid typo. Thanks for fixing the test!
> 
> Are you still looking at this, or did you forget to lgtm?

The latter :). LGTM!

[achuithb, 2015-09-09 09:06:08]

On 2015/09/09 09:04:58, bartfab wrote:
> On 2015/09/09 08:50:31, achuithb wrote:
> > On 2015/09/08 14:57:17, bartfab wrote:
> > >
> >
>
https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
> > > File chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
> > > (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/1319373003/diff/1/chrome/browser/chromeos/log...
> > > chrome/browser/chromeos/login/existing_user_controller_browsertest.cc:463:
> > > runner2 = new content::MessageLoopRunner;
> > > Oops, that was quite a stupid typo. Thanks for fixing the test!
> > 
> > Are you still looking at this, or did you forget to lgtm?
> 
> The latter :). LGTM!

Ty :)

[commit-bot: I haz the power, 2015-09-09 09:24:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-09 09:24:56]

Dry run: This issue passed the CQ dry run.

[achuithb, 2015-09-09 09:40:09]

The CQ bit was checked by achuith@chromium.org

[commit-bot: I haz the power, 2015-09-09 09:40:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1319373003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1319373003/1

[commit-bot: I haz the power, 2015-09-09 10:06:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-09 10:07:14]

Patchset 1 (id:??) landed as
https://crrev.com/f189d5356a18e7b5e159c8028184137eb8656e06
Cr-Commit-Position: refs/heads/master@{#347898}