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

Issue 797746 link

Starred by 3 users
Status: Verified
Owner:
fukino@chromium.org
Closed: Jan 2018
Cc:
ajha@chromium.org
█ oka@chromium.org
█ brajkumar@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Wrong Duration of Video is seen at 'General Info' details in General information page of Files App

Project Member Reported by mmanchala@chromium.org, Dec 27 2017 
Chrome Version: 65.0.3299.0/10253.0.0 Peppy,Candy,Gnawty
OS: Chrome

Sample URL : http://www.sample-videos.com/

What steps will reproduce the problem?
(1)Sign in to user ->Now go to above URL -> download any file(Ex: Download 1Mb,2Mb,5Mb,10Mb,20Mb videos)
(2)Now go to Files App  -> Select downloaded video -> Right Click and select 'Get info' option or click on 'Space' Key
(3)Now in General information page observe 'Duration' of video which is available at 'General Info' details i.e. Duration of downloaded Video will be '0:13' sec and at 'General Info' details Duration of Video is '0:04'
(Please refer Video and Screenshot)

Expected: Correct Duration of video should be seen at 'General Info' details in General information page 
Actual: Instead Wrong Duration is seen at 'General Info' details

This is Regression Issue seen as same is working fine in M-61

@fukino : Please confirm the Issue
Actual_Duration.webm
7.7 MB View Download
Actual_Duration.jpg
172 KB View Download
Expected_Duration.jpg
155 KB View Download
Expected_Duration.webm
4.5 MB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 16 2018 

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

commit a6c839839e82ddde98b667756cf38c0b8e37b790
Author: Naoki Fukino <fukino@chromium.org>
Date: Tue Jan 16 09:14:26 2018

Files app: Fix video duration on quick view metadata info.

To show video's duration on quick view, the string for duration is constructed
assuming that the given duration in seconds is integer.
However, it can be decimal number. We need to convert it to integer before
costructing the string.

Bug:  797746 
Test: Manually tested
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I83fbf0de491cc8700b5cb7222344cdf9946ecfae
Reviewed-on: https://chromium-review.googlesource.com/867145
Reviewed-by: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Commit-Queue: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529386}
[modify] https://crrev.com/a6c839839e82ddde98b667756cf38c0b8e37b790/ui/file_manager/file_manager/foreground/elements/files_metadata_box.js

Comment 2 by fukino@chromium.org, Jan 16 2018

Status: Fixed (was: Assigned)

Comment 3 by mkarkada@chromium.org, Jan 26 2018

Status: Verified (was: Fixed)
Verified on M65- 10323.4.0, 65.0.3325.16 dev build.
►

Sign in to add a comment