New issue
Advanced search Search tips

Issue 674549 link

Starred by 2 users
Status: Available
Owner: ----
Cc:
lfg@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 674507



Sign in to add a comment

ArrayBufferView should be a union type (typedef), not an interface

Project Member Reported by foolip@chromium.org, Dec 15 2016 
https://heycam.github.io/webidl/#ArrayBufferView

Blink's ArrayBufferView.idl instead has:

[
    ImplementedAs=DOMArrayBufferView,
    NoInterfaceObject,
] interface ArrayBufferView {
    readonly attribute ArrayBuffer buffer;
    readonly attribute unsigned long byteOffset;
    readonly attribute unsigned long byteLength;
};

Found because of the [NoInterfaceObject] bit.

Comment 1 by yukishiino@chromium.org, Jan 30 2017 

The actual implementation of ArrayBufferView is provided with v8::ArrayBufferView, and Blink provides almost nothing.  The content of ArrayBufferView.idl is mostly dummy.  Should we just remove the .idl file?  Or, do we want the typedef declaration in it?

Comment 2 by foolip@chromium.org, Jan 30 2017 

I think it would be great if we could remove these IDL files since they aren't actually what's used for the implementation.

Comment 3 by peria@chromium.org, Mar 2 2017

Status: Available (was: Untriaged)
Project Member

Comment 4 by sheriffbot@chromium.org, Mar 8 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by peria@chromium.org, Mar 11 2018

Status: Available (was: Untriaged)
►

Sign in to add a comment