New issue
Advanced search Search tips

Issue 695972 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 752877

Blocking:
issue 674593


Show other hotlists

Hotlists containing this issue:
Non-Standard-IDL


Sign in to add a comment

RadioNodeList#getter should be inherited from NodeList

Project Member Reported by lunalu@chromium.org, Feb 24 2017

Issue description

As titled.
 

Comment 1 by foolip@chromium.org, Feb 27 2017

Cc: bashi@chromium.org
This could be another case where it should be made virtual in C++ instead of using override in IDL, but given that the getter is anonymous (not named), I think that it's impossible to observe the existence of the getter from JavaScript. bashi@, can you confirm?

(With named getters, attributes and methods, one can use getOwnPropertyDescriptor on the prototype of the parent interface, and use that to run the parent interfaces's getters or methods on an instance of a derived interface which overrides the same thing. An odd little thing that makes these situations interesting.)

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

Components: Blink>Bindings
Status: Available (was: Untriaged)

Comment 3 by bashi@chromium.org, Mar 3 2017

Cc: yukishiino@chromium.org
Sorry I missed this.

Yeah, it seems that the code generator doesn't check the existence of getters in parent interfaces. Maybe we need something similar to inherit_unforgeable_attributes()

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/bindings/scripts/interface_dependency_resolver.py?q=inherit_unforgeable_attributes&l=319

Comment 4 by tkent@chromium.org, Aug 7 2017

Blockedon: 752877
Project Member

Comment 5 by sheriffbot@chromium.org, Aug 7

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.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -bashi@chromium.org
Status: Available (was: Untriaged)
Owner: peria@chromium.org
Status: Assigned (was: Available)
Cc: peria@chromium.org
 Issue 752877  has been merged into this issue.

Sign in to add a comment