Fix the data race of OobeTest.NewUser browser test and enable it.

Fix the data race of OobeTest.NewUser browser test and enable it.

BUG=178009
TEST=for i in {1..20}; do \
     out/Debug/interactive_ui_tests --gtest_filter=OobeTest.NewUser --disable-test-compositor; \
     if [ $? != 0 ]; then break; fi; \
     done;

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

[glotov, 2013-04-18 16:01:28]

Hi!

[Nikita (slow), 2013-04-19 07:34:06]

lgtm

https://codereview.chromium.org/13895008/diff/19001/chrome/browser/chromeos/l...
File chrome/browser/chromeos/login/oobe_browsertest.cc (right):

https://codereview.chromium.org/13895008/diff/19001/chrome/browser/chromeos/l...
chrome/browser/chromeos/login/oobe_browsertest.cc:211: // Temporally disabled
because of a race: crbug.com/178009
Is this still disabled?

[kjellander_chromium, 2013-04-19 07:38:59]

https://codereview.chromium.org/13895008/diff/19001/chrome/browser/chromeos/l...
File chrome/browser/chromeos/login/oobe_browsertest.cc (right):

https://codereview.chromium.org/13895008/diff/19001/chrome/browser/chromeos/l...
chrome/browser/chromeos/login/oobe_browsertest.cc:211: // Temporally disabled
because of a race: crbug.com/178009
On 2013/04/19 07:34:07, Nikita Kostylev wrote:
> Is this still disabled?

Confusing. I guess this CL needs a rebase and then remove this comment if it's
re-enabling the test.

[commit-bot: I haz the power, 2013-04-19 09:20:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/glotov@chromium.org/13895008/29001

[glotov, 2013-04-19 09:21:12]

https://codereview.chromium.org/13895008/diff/19001/chrome/browser/chromeos/l...
File chrome/browser/chromeos/login/oobe_browsertest.cc (right):

https://codereview.chromium.org/13895008/diff/19001/chrome/browser/chromeos/l...
chrome/browser/chromeos/login/oobe_browsertest.cc:211: // Temporally disabled
because of a race: crbug.com/178009
On 2013/04/19 07:38:59, Henrik Kjellander wrote:
> On 2013/04/19 07:34:07, Nikita Kostylev wrote:
> > Is this still disabled?
> 
> Confusing. I guess this CL needs a rebase and then remove this comment if it's
> re-enabling the test.

Done.

[commit-bot: I haz the power, 2013-04-19 13:17:56]

Change committed as 195157