thumbnail generation prevents video playback |
||||||||||
Issue descriptionChrome Version : 49.0.2623.95 OS Version: 7834.60.0 What steps will reproduce the problem? 1. Insert SD card, browse to directory with multiple large videos. eg. 15 ~280 mb MOV files. 2. Double click on a video. What is the expected result? Video plays immediately. What happens instead of that? Black video player opens. Nothing happens for several minutes. Thumbnails of videos in the folder start to appear very slowly. Please provide any additional information below. Attach a screenshot if possible. UserAgentString: Mozilla/5.0 (X11; CrOS x86_64 7834.60.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.95 Safari/537.36
,
Mar 30 2016
#CBC-RS/TC-watchlist
,
Mar 30 2016
Confirmed, with the additional information that it isn't even necessary to open a video; as soon as the storage device is mounted, the Files app starts chewing up memory to render thumbnails. Issue reported @ ~0940 EDT on March 30, 2016. Further details, screenshots: https://productforums.google.com/d/msg/chromebook-central/IfU4Si8rd6g/j4Juj8keCQAJ 2012 Pixel with 4 Gb of RAM, running Version 49.0.2623.95 (64-bit) Platform 7834.60.0 (Official Build) stable-channel link Firmware Google_Link.2695.1.169
,
Mar 31 2016
,
Mar 31 2016
It looks toDataURL blocks file opening.
,
Mar 31 2016
Issue 598673 has been merged into this issue.
,
Apr 1 2016
It found chrome generated 5GB GIF files under /tmp.
,
Apr 1 2016
5GB gif was another problem. created the fix for this issue. crrev.com/1842313005
,
Apr 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5d9ebe5481791288b7dd28bc51aa608de3e101a9 commit 5d9ebe5481791288b7dd28bc51aa608de3e101a9 Author: hirono <hirono@chromium.org> Date: Fri Apr 01 08:02:31 2016 Don't load all bytes of video when generating thumbnails. Previously ImageLoader loaded all bytes of video by using XHR before generating thumbnails. The CL fixes it so that ImageLoader assign entry URL to video tag directly, which avoids from loading all bytes to memory. BUG= 597243 TEST=manually Review URL: https://codereview.chromium.org/1842313005 Cr-Commit-Position: refs/heads/master@{#384515} [modify] https://crrev.com/5d9ebe5481791288b7dd28bc51aa608de3e101a9/ui/file_manager/image_loader/request.js
,
Apr 1 2016
,
Apr 1 2016
,
Apr 2 2016
Your change meets the bar and is auto-approved for M50 (branch: 2661)
,
Apr 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/72d9e1dfd3164a33424de724465449924dc6dbf3 commit 72d9e1dfd3164a33424de724465449924dc6dbf3 Author: Daichi Hirono <hirono@chromium.org> Date: Mon Apr 04 03:54:29 2016 Don't load all bytes of video when generating thumbnails. Previously ImageLoader loaded all bytes of video by using XHR before generating thumbnails. The CL fixes it so that ImageLoader assign entry URL to video tag directly, which avoids from loading all bytes to memory. BUG= 597243 TEST=manually Review URL: https://codereview.chromium.org/1842313005 Cr-Commit-Position: refs/heads/master@{#384515} (cherry picked from commit 5d9ebe5481791288b7dd28bc51aa608de3e101a9) Review URL: https://codereview.chromium.org/1855143002 . Cr-Commit-Position: refs/branch-heads/2661@{#469} Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081} [modify] https://crrev.com/72d9e1dfd3164a33424de724465449924dc6dbf3/ui/file_manager/image_loader/request.js
,
Apr 6 2016
Issue 600414 has been merged into this issue.
,
Aug 29 2016
,
Aug 31 2016
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by mtomasz@chromium.org
, Mar 24 2016Owner: hirono@chromium.org
Status: Assigned (was: Unconfirmed)