Fix copy number counting in Files.app.

Fix copy number counting in Files.app.

BUG=343835
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251988

[hirono, 2014-02-14 10:56:39]

Thank you for fixing it!

https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
File
chrome/browser/resources/file_manager/background/js/file_operation_manager.js
(right):

https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
chrome/browser/resources/file_manager/background/js/file_operation_manager.js:639:
// Finalize the entry's progress state.
Could you please remove line 639 ~ 642 while you are here. This is not used.

https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
chrome/browser/resources/file_manager/background/js/file_operation_manager.js:645:
this.processedBytes = this.calcProcessedBytes_();
Could you please also move the line to the complete callback?

[kinaba, 2014-02-19 00:34:35]

https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
File
chrome/browser/resources/file_manager/background/js/file_operation_manager.js
(right):

https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
chrome/browser/resources/file_manager/background/js/file_operation_manager.js:639:
// Finalize the entry's progress state.
On 2014/02/14 10:56:39, hirono wrote:
> Could you please remove line 639 ~ 642 while you are here. This is not used.

Done.

https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
chrome/browser/resources/file_manager/background/js/file_operation_manager.js:645:
this.processedBytes = this.calcProcessedBytes_();
On 2014/02/14 10:56:39, hirono wrote:
> Could you please also move the line to the complete callback?

Done.

[hirono, 2014-02-19 02:13:11]

On 2014/02/19 00:34:35, kinaba (ooo until Feb 18) wrote:
>
https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
> File
> chrome/browser/resources/file_manager/background/js/file_operation_manager.js
> (right):
> 
>
https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
>
chrome/browser/resources/file_manager/background/js/file_operation_manager.js:639:
> // Finalize the entry's progress state.
> On 2014/02/14 10:56:39, hirono wrote:
> > Could you please remove line 639 ~ 642 while you are here. This is not used.
> 
> Done.
> 
>
https://codereview.chromium.org/166263002/diff/20002/chrome/browser/resources...
>
chrome/browser/resources/file_manager/background/js/file_operation_manager.js:645:
> this.processedBytes = this.calcProcessedBytes_();
> On 2014/02/14 10:56:39, hirono wrote:
> > Could you please also move the line to the complete callback?
> 
> Done.

lgtm, thanks!

[kinaba, 2014-02-19 02:14:03]

The CQ bit was checked by kinaba@chromium.org

[commit-bot: I haz the power, 2014-02-19 02:16:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/166263002/190001

[commit-bot: I haz the power, 2014-02-19 08:04:04]

Change committed as 251988