Avoid inserting duplicated entries after creating a directory.

Avoid inserting duplicated entries after creating a directory.

New entries which are passed to DirectoryContents.onNewEntries_ are determined in DorectoryContents.update, but before they are inserted in the file list, the file list may change.
When creating a new directory in Drive, this sometimes occurs.
This CL adds duplication check just before inserting entries.

BUG=423646
TEST=creating a directory in MyDrive By Ctrl-E and check if there's no duplicated folder.

Committed: https://crrev.com/2a724e66aff9c867195b21fd774fa563e315e1e2
Cr-Commit-Position: refs/heads/master@{#301335}

[fukino, 2014-10-27 05:09:49]

fukino@chromium.org changed reviewers:
+ yoshiki@chromium.org

[fukino, 2014-10-27 05:09:49]

yoshiki@, could you take a look? Thanks!

https://codereview.chromium.org/676293002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/directory_contents.js (right):

https://codereview.chromium.org/676293002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/directory_contents.js:831:
currentURLs[this.fileList_.item(i).toURL()] = true;
This duplication check add some cost, but this doesn't change asymptotic
complexity as the following this.fileList_.push() already takes O(N), where N is
the length of the file list.

[yoshiki, 2014-10-27 08:53:17]

lgtm

[fukino, 2014-10-27 09:22:42]

On 2014/10/27 08:53:17, yoshiki wrote:
> lgtm

Thanks!

[fukino, 2014-10-27 09:22:45]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-10-27 09:23:48]

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

[commit-bot: I haz the power, 2014-10-27 10:09:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-27 10:10:11]

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