1) Add a drop down revealing a popup panel with import details (with is a WIP, but the current state should be sufficient to land).

2) Update scanner to publish updates on all scan activity (so the details reflects valid information while scanning). In doing this I pulled all of the deduping logic up out of ScanResult into MediaScanner...improving separation of concern between the classes.

3) Rework update model to eliminate a flicker resulting from the brief update to show zero results when a file selection is transitioning from n to n+/-1.

Sadly, this change results in a net loss of readability, especially WRT having good clear decision points. I've added comments to help out in this respect.

FYI, I'm OOO for the rest of the day. I think this CL is good to land, rough though it may be.

In a followup CL:

1) I'll make some subsequent improvements to unittests in the wake of removing the "GetCommandUpdate" coverage (removed because GetCommandUpdate exists no more).

2) This change still exhibits one odd behavior, ignoring entirely a selection when it consistes of all deduped files...in that case we *do* want to see zero results. Will address at the time of test improvements.

BUG=420680
TEST=browser_test: FileManagerJsTest.*

Review URL: https://codereview.chromium.org/899943002

Cr-Commit-Position: refs/heads/master@{#315094}

0 files changed, 0 insertions, 0 deletions