New issue
Advanced search Search tips

Issue 887593 link

Starred by 2 users
Status: Fixed
Owner: ----
Closed: Sep 25
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Video Decoded Counts are too narrow

Project Member Reported by dtapu...@chromium.org, Sep 20 
The video and audio decoded counts are
unsigned longs in the IDL.

They really should be unsigned long longs.

    // Non-standard APIs
    // The number of bytes consumed by the media decoder.
    [MeasureAs=PrefixedAudioDecodedByteCount] readonly attribute unsigned long webkitAudioDecodedByteCount;
    [MeasureAs=PrefixedVideoDecodedByteCount] readonly attribute unsigned long webkitVideoDecodedByteCount;

Comment 1 Deleted

Comment 2 by sapz...@gmail.com, Sep 22 

I'm interested in this issue, too.
Can i take this issue if you don't mind?
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 25 

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

commit 129655ca027dd2c11864f1e5080c89109eac36ba
Author: Jongheon Kim <sapzape@gmail.com>
Date: Tue Sep 25 16:28:40 2018

Change data type unsigned longs to unsigned long longs

This patch changes the data type of video and audio decode counts
in the IDL

Bug:  887593 
Change-Id: Ib988825d6db4eb7b1d18a3d2b5d2b4c2b791e882
Reviewed-on: https://chromium-review.googlesource.com/1240573
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593963}
[modify] https://crrev.com/129655ca027dd2c11864f1e5080c89109eac36ba/third_party/blink/renderer/core/html/media/html_media_element.cc
[modify] https://crrev.com/129655ca027dd2c11864f1e5080c89109eac36ba/third_party/blink/renderer/core/html/media/html_media_element.h
[modify] https://crrev.com/129655ca027dd2c11864f1e5080c89109eac36ba/third_party/blink/renderer/core/html/media/html_media_element.idl

Comment 4 by dtapu...@chromium.org, Sep 25

Status: Fixed (was: Untriaged)
►

Sign in to add a comment