New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 712547 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

No UI feedback for a long time if you copy many files from MTP

Project Member Reported by satorux@chromium.org, Apr 18 2017

Issue description


What steps will reproduce the problem?
1. Select ~2000 files in a MTP folder
2. Press Ctrl-C
3. Create a new Files app window
4. Press Ctrl-V in Downloads

What is the expected result?

UI responds fast to Ctrl-V

What happens instead of that?

I took about 30 seconds before the copy progress appears at the left bottom of Files app window

Please provide any additional information below. Attach a screenshot if
possible.

UserAgentString: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36



 

Comment 1 by tetsui@chromium.org, Apr 21 2017

Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eab6fcca575e4781af96c9f1b81265e05f370744

commit eab6fcca575e4781af96c9f1b81265e05f370744
Author: tetsui <tetsui@google.com>
Date: Fri Apr 28 02:12:48 2017

Update progress center immediately when files are pasted.

URLsToEntriesWithAccess is heavy operation especially on slow storage
devices such as MTP connected ones.
Previously, progress center was updated only after
URLsToEntriesWithAccess was finished. As a result, it took more than 30
seconds for a user to receive feedback in some cases.

BUG= 712547 
TEST=manually tested.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2846763002
Cr-Commit-Position: refs/heads/master@{#467840}

[modify] https://crrev.com/eab6fcca575e4781af96c9f1b81265e05f370744/ui/file_manager/file_manager/foreground/js/file_transfer_controller.js

Comment 3 by tetsui@chromium.org, Apr 28 2017

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
ChromeOS 9557.0.0, 60.0.3101.0

Comment 5 by tetsui@chromium.org, Jun 27 2017

On ToT (c1f83c1e9c3eb2dcf4a8d07a60c0753fd7dba5ea) it seems the feedback became very slow again even though the CL in #2 is still effective. I would investigate that.

Sign in to add a comment