Add scan mode to choose whether to do content scan or only history check.

Add scan mode parameter to choose whether to do content duplication check or just check the import history.

This change will not actually change the behavior of the app but will be used by a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/4703c1fc80d263275181e92c9bd1e124036f4e64
Cr-Commit-Position: refs/heads/master@{#439422}

[yamaguchi, 2016-12-13 09:03:35]

Description was changed from

==========
Add scan mode to choose whether to do content scan or only history check.

BUG=
==========

to

==========
Add scan mode to choose whether to do content scan or only history check.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[yamaguchi, 2016-12-13 09:22:31]

The CQ bit was checked by yamaguchi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 09:22:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yamaguchi, 2016-12-13 09:45:58]

Description was changed from

==========
Add scan mode to choose whether to do content scan or only history check.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the code but will be used
in a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[yamaguchi, 2016-12-13 09:46:11]

The CQ bit was checked by yamaguchi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 09:46:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yamaguchi, 2016-12-13 09:47:53]

Description was changed from

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the code but will be used
in a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the app but will be used by
a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[yamaguchi, 2016-12-13 09:47:53]

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

[commit-bot: I haz the power, 2016-12-13 10:13:47]

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

[commit-bot: I haz the power, 2016-12-13 10:13:48]

Dry run: This issue passed the CQ dry run.

[fukino, 2016-12-16 05:07:41]

Sorry for a late response.

https://codereview.chromium.org/2568723005/diff/20001/ui/file_manager/file_ma...
File ui/file_manager/file_manager/foreground/js/import_controller.js (right):

https://codereview.chromium.org/2568723005/diff/20001/ui/file_manager/file_ma...
ui/file_manager/file_manager/foreground/js/import_controller.js:295: var scan =
this.scanManager_.getActiveScan();
This scan object is not used.
Do we need to keep this line? (Is the side effect of getActiveScan() needed?)

[yamaguchi, 2016-12-16 06:52:37]

https://codereview.chromium.org/2568723005/diff/20001/ui/file_manager/file_ma...
File ui/file_manager/file_manager/foreground/js/import_controller.js (right):

https://codereview.chromium.org/2568723005/diff/20001/ui/file_manager/file_ma...
ui/file_manager/file_manager/foreground/js/import_controller.js:295: var scan =
this.scanManager_.getActiveScan();
On 2016/12/16 05:07:41, fukino wrote:
> This scan object is not used.
> Do we need to keep this line? (Is the side effect of getActiveScan() needed?)

Sorry, the change was included by a simple mistake when I split this change out
of a larger change.

[fukino, 2016-12-19 06:24:54]

lgtm

[yamaguchi, 2016-12-19 06:41:18]

The CQ bit was checked by yamaguchi@chromium.org

[commit-bot: I haz the power, 2016-12-19 06:41:26]

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, 2016-12-19 07:05:51]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1482129678138650,
"parent_rev": "6d6404c18a24a73a6e2fe7b68a8f170b25bf3178", "commit_rev":
"f35547712b90637c661c4456644827b76b9524e1"}

[commit-bot: I haz the power, 2016-12-19 07:06:21]

Description was changed from

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the app but will be used by
a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the app but will be used by
a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2568723005
==========

[commit-bot: I haz the power, 2016-12-19 07:06:22]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-19 07:08:25]

Description was changed from

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the app but will be used by
a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2568723005
==========

to

==========
Add scan mode parameter to choose whether to do content duplication check or
just check the import history.

This change will not actually change the behavior of the app but will be used by
a following change https://codereview.chromium.org/2569163002/ .

BUG=668574
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/4703c1fc80d263275181e92c9bd1e124036f4e64
Cr-Commit-Position: refs/heads/master@{#439422}
==========

[commit-bot: I haz the power, 2016-12-19 07:08:26]

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