Remove races in window creation in Files.app.

Remove races in window creation in Files.app.

This patch removes races in case of requested reusing of windows. It might happend, that windows are not yet fully initialized and we tried to reuse them. This bug is fixed by waiting for all windows to be initialized by enqueuing the reusing logic.

TEST=Follow the bug's repro steps.
BUG=238412
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202011

[mtomasz, 2013-05-23 06:33:01]

@yoshiki: PTAL. Thanks.

[yoshiki, 2013-05-23 06:49:21]

https://codereview.chromium.org/15817002/diff/2001/chrome/browser/resources/f...
File chrome/browser/resources/file_manager/js/background.js (right):

https://codereview.chromium.org/15817002/diff/2001/chrome/browser/resources/f...
chrome/browser/resources/file_manager/js/background.js:351: });
In test_util.js, the retual value is used. Could you modify test_util.js?

[mtomasz, 2013-05-23 06:50:41]

On 2013/05/23 06:49:21, yoshiki wrote:
>
https://codereview.chromium.org/15817002/diff/2001/chrome/browser/resources/f...
> File chrome/browser/resources/file_manager/js/background.js (right):
> 
>
https://codereview.chromium.org/15817002/diff/2001/chrome/browser/resources/f...
> chrome/browser/resources/file_manager/js/background.js:351: });
> In test_util.js, the retual value is used. Could you modify test_util.js?

Oops! Good catch.

[mtomasz, 2013-05-23 07:17:30]

https://codereview.chromium.org/15817002/diff/2001/chrome/browser/resources/f...
File chrome/browser/resources/file_manager/js/background.js (right):

https://codereview.chromium.org/15817002/diff/2001/chrome/browser/resources/f...
chrome/browser/resources/file_manager/js/background.js:351: });
On 2013/05/23 06:49:21, yoshiki wrote:
> In test_util.js, the retual value is used. Could you modify test_util.js?

Done.

[yoshiki, 2013-05-23 07:24:29]

lgtm

[commit-bot: I haz the power, 2013-05-23 07:35:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/15817002/8001

[commit-bot: I haz the power, 2013-05-23 07:48:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/15817002/16001

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

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-05-23 11:10:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/15817002/22001

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

Retried try job too often on win7_aura for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-05-24 03:22:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/15817002/22001

[commit-bot: I haz the power, 2013-05-24 07:38:18]

Change committed as 202011