Serialize reading and writing in the import history storage.

Serialize reading and writing in the import history storage.

Concurrently writing with multiple FileWriters causes broken files. This patch
wraps both reading a writing with a queue made of chained promises.

TEST=browser_tests: *ImportHistoryTest*
BUG=None

Committed: https://crrev.com/c51c835b762f3bd0c47b2b7ea47450f2ac1abfe6
Cr-Commit-Position: refs/heads/master@{#312999}

[mtomasz, 2015-01-23 22:35:37]

mtomasz@chromium.org changed reviewers:
+ smckay@google.com

[mtomasz, 2015-01-23 22:35:46]

@smckay: PTAL.

[Steve McKay, 2015-01-23 23:03:52]

smckay@chromium.org changed reviewers:
+ smckay@chromium.org

[Steve McKay, 2015-01-23 23:03:52]

Neat trick! I like it.

LGTM

[mtomasz, 2015-01-23 23:05:24]

The CQ bit was checked by mtomasz@chromium.org

[hirono, 2015-01-23 23:05:58]

hirono@chromium.org changed reviewers:
+ hirono@chromium.org

[hirono, 2015-01-23 23:05:58]

https://codereview.chromium.org/868133004/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/background/js/import_history.js (right):

https://codereview.chromium.org/868133004/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/background/js/import_history.js:751:
.then(this.writeRecord_.bind(this, record));
We should catch rejection here. Otherwise the next operation will be skipped.

[commit-bot: I haz the power, 2015-01-23 23:06:21]

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

[mtomasz, 2015-01-23 23:08:41]

The CQ bit was unchecked by mtomasz@chromium.org

[mtomasz, 2015-01-23 23:12:04]

https://codereview.chromium.org/868133004/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/background/js/import_history.js (right):

https://codereview.chromium.org/868133004/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/background/js/import_history.js:751:
.then(this.writeRecord_.bind(this, record));
On 2015/01/23 23:05:58, hirono wrote:
> We should catch rejection here. Otherwise the next operation will be skipped.

Good point. Done.

[mtomasz, 2015-01-23 23:12:15]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2015-01-23 23:13:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/868133004/20001

[mtomasz, 2015-01-24 00:19:15]

mtomasz@chromium.org changed reviewers:
- smckay@google.com

[commit-bot: I haz the power, 2015-01-24 00:23:08]

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

[commit-bot: I haz the power, 2015-01-24 00:23:09]

Try jobs failed on following builders:
  linux_chromium_asan_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mtomasz, 2015-01-24 00:30:59]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2015-01-24 00:31:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/868133004/20001

[commit-bot: I haz the power, 2015-01-24 00:32:16]

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

[commit-bot: I haz the power, 2015-01-24 00:32:17]

Try jobs failed on following builders:
  linux_chromium_asan_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mtomasz, 2015-01-24 01:47:16]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2015-01-24 01:48:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/868133004/40001

[commit-bot: I haz the power, 2015-01-24 02:37:55]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-01-24 02:38:51]

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