Revert of Reuse FileTasks when entries are not changed.

Revert of Reuse FileTasks when entries are not changed. (patchset #3 id:40001 of https://codereview.chromium.org/2833413003/ )

Reason for revert:
We'd like to revert this patch because this seem to have caused regression crbug.com/738803.
2973003002 needs to be merged before this CL in order to revert another dependent change first.

Original issue's description:
> Reuse FileTasks when entries are not changed.
> 
> fileManagerPrivate.getFileTasks is slow.
> However, every time a file is copied during multi file copy, the method
> was called via CHANGE_THROTTLED event listener.
> This CL stops creating FileTasks again when file entries for the
> previous FileTasks are same as the current ones.
> Other possible solution to this problem is to change C++ interface of
> fileManagerPrivate.getFileTasks to support incremental update of
> FileTasks.
> 
> BUG=462989, 712121
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Review-Url: https://codereview.chromium.org/2833413003
> Cr-Commit-Position: refs/heads/master@{#467264}
> Committed: https://chromium.googlesource.com/chromium/src/+/6e537c987f4d8cc6fcfda8f80b4c3216663371d4

TBR=fukino@chromium.org,tetsui@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=462989, 712121, 738803
Review-Url: https://codereview.chromium.org/2966163005
Cr-Commit-Position: refs/heads/master@{#484512}
Committed: https://chromium.googlesource.com/chromium/src/+/5474326ed3852a9e39a625af8c3b4f1b46ac7854

[yamaguchi, 2017-07-06 04:47:18]

Created Revert of Reuse FileTasks when entries are not changed.

[tetsui, 2017-07-06 07:57:11]

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

[tetsui, 2017-07-06 07:57:11]

lgtm

[yamaguchi, 2017-07-06 08:19:30]

Description was changed from

==========
Revert of Reuse FileTasks when entries are not changed. (patchset #3 id:40001 of
https://codereview.chromium.org/2833413003/ )

Reason for revert:
We'd like to revert this patch because this seem to have caused regression
crbug.com/738803.
2973003002 needs to be merged before this CL in order to revert another
dependent change first.

Original issue's description:
> Reuse FileTasks when entries are not changed.
> 
> fileManagerPrivate.getFileTasks is slow.
> However, every time a file is copied during multi file copy, the method
> was called via CHANGE_THROTTLED event listener.
> This CL stops creating FileTasks again when file entries for the
> previous FileTasks are same as the current ones.
> Other possible solution to this problem is to change C++ interface of
> fileManagerPrivate.getFileTasks to support incremental update of
> FileTasks.
> 
> BUG=462989,712121
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Review-Url: https://codereview.chromium.org/2833413003
> Cr-Commit-Position: refs/heads/master@{#467264}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6e537c987f4d8cc6fcfda8f80b4c...

TBR=fukino@chromium.org,tetsui@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=462989,712121
==========

to

==========
Revert of Reuse FileTasks when entries are not changed. (patchset #3 id:40001 of
https://codereview.chromium.org/2833413003/ )

Reason for revert:
We'd like to revert this patch because this seem to have caused regression
crbug.com/738803.
2973003002 needs to be merged before this CL in order to revert another
dependent change first.

Original issue's description:
> Reuse FileTasks when entries are not changed.
> 
> fileManagerPrivate.getFileTasks is slow.
> However, every time a file is copied during multi file copy, the method
> was called via CHANGE_THROTTLED event listener.
> This CL stops creating FileTasks again when file entries for the
> previous FileTasks are same as the current ones.
> Other possible solution to this problem is to change C++ interface of
> fileManagerPrivate.getFileTasks to support incremental update of
> FileTasks.
> 
> BUG=462989,712121
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Review-Url: https://codereview.chromium.org/2833413003
> Cr-Commit-Position: refs/heads/master@{#467264}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6e537c987f4d8cc6fcfda8f80b4c...

TBR=fukino@chromium.org,tetsui@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=462989,712121,738803
==========

[tetsui, 2017-07-06 08:53:50]

The CQ bit was checked by tetsui@chromium.org

[commit-bot: I haz the power, 2017-07-06 08:54:02]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yamaguchi, 2017-07-06 08:59:03]

The CQ bit was unchecked by yamaguchi@chromium.org

[yamaguchi, 2017-07-06 08:59:10]

The CQ bit was checked by yamaguchi@chromium.org

[commit-bot: I haz the power, 2017-07-06 08:59:20]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-06 09:24:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499331550673120, "parent_rev":
"0e1d5a7f990f06911ea848d97f4373619e2a35c7", "commit_rev":
"5474326ed3852a9e39a625af8c3b4f1b46ac7854"}

[commit-bot: I haz the power, 2017-07-06 09:24:33]

Description was changed from

==========
Revert of Reuse FileTasks when entries are not changed. (patchset #3 id:40001 of
https://codereview.chromium.org/2833413003/ )

Reason for revert:
We'd like to revert this patch because this seem to have caused regression
crbug.com/738803.
2973003002 needs to be merged before this CL in order to revert another
dependent change first.

Original issue's description:
> Reuse FileTasks when entries are not changed.
> 
> fileManagerPrivate.getFileTasks is slow.
> However, every time a file is copied during multi file copy, the method
> was called via CHANGE_THROTTLED event listener.
> This CL stops creating FileTasks again when file entries for the
> previous FileTasks are same as the current ones.
> Other possible solution to this problem is to change C++ interface of
> fileManagerPrivate.getFileTasks to support incremental update of
> FileTasks.
> 
> BUG=462989,712121
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Review-Url: https://codereview.chromium.org/2833413003
> Cr-Commit-Position: refs/heads/master@{#467264}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6e537c987f4d8cc6fcfda8f80b4c...

TBR=fukino@chromium.org,tetsui@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=462989,712121,738803
==========

to

==========
Revert of Reuse FileTasks when entries are not changed. (patchset #3 id:40001 of
https://codereview.chromium.org/2833413003/ )

Reason for revert:
We'd like to revert this patch because this seem to have caused regression
crbug.com/738803.
2973003002 needs to be merged before this CL in order to revert another
dependent change first.

Original issue's description:
> Reuse FileTasks when entries are not changed.
> 
> fileManagerPrivate.getFileTasks is slow.
> However, every time a file is copied during multi file copy, the method
> was called via CHANGE_THROTTLED event listener.
> This CL stops creating FileTasks again when file entries for the
> previous FileTasks are same as the current ones.
> Other possible solution to this problem is to change C++ interface of
> fileManagerPrivate.getFileTasks to support incremental update of
> FileTasks.
> 
> BUG=462989,712121
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> 
> Review-Url: https://codereview.chromium.org/2833413003
> Cr-Commit-Position: refs/heads/master@{#467264}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6e537c987f4d8cc6fcfda8f80b4c...

TBR=fukino@chromium.org,tetsui@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=462989,712121,738803

Review-Url: https://codereview.chromium.org/2966163005
Cr-Commit-Position: refs/heads/master@{#484512}
Committed:
https://chromium.googlesource.com/chromium/src/+/5474326ed3852a9e39a625af8c3b...
==========

[commit-bot: I haz the power, 2017-07-06 09:24:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5474326ed3852a9e39a625af8c3b...