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

Issue 731484 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug
mtp



Sign in to add a comment

Quick view on a folder flickers while copying is in progress

Project Member Reported by satorux@chromium.org, Jun 9 2017

Issue description

Chrome Version       : Almost ToT (crrev.com/477915)

What steps will reproduce the problem?
1. Press Ctrl-C on a folder with ~2000 photos on a MTP device 
2. Press Ctrl-V on a USB flash drive
3. Select the destination folder and open the quick view with SPACE key

What is the expected result?

The quick view appears and does not flicker

What happens instead of that?

The quick view appears and the value of the Size field flickers between two states:

1) the value is shown (ex. 8.2 GB)
2) the value is not shown
 

Comment 1 by tetsui@chromium.org, Jun 14 2017

Cc: -tetsui@chromium.org oka@chromium.org
Owner: tetsui@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 14 2017

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

commit f34801c28ded7c524030c5fe565e9ac420c4ce28
Author: tetsui <tetsui@chromium.org>
Date: Wed Jun 14 09:13:24 2017

Keep previous size in Directory QuickView.

In QuickView of a directory, size field was flickering becuase every
time the directory is updated, the size field was hidden until its size
is retrieved from fileManagerPrivate.getDirectorySize.
Now it show previous directory size until the new size is retrieved if
the selected directory is not changed.

BUG= 731484 , 733143 
TEST=manual
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/f34801c28ded7c524030c5fe565e9ac420c4ce28/ui/file_manager/file_manager/foreground/elements/files_metadata_box.js
[modify] https://crrev.com/f34801c28ded7c524030c5fe565e9ac420c4ce28/ui/file_manager/file_manager/foreground/js/metadata_box_controller.js

Comment 3 by tetsui@chromium.org, Jun 19 2017

Status: Fixed (was: Started)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment