New issue
Advanced search Search tips

Issue 674520 link

Starred by 8 users

Issue metadata

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

Blocking:
issue 612962
issue 673664



Sign in to add a comment

ResizeObserverEntry has a constructor in the spec

Project Member Reported by foolip@chromium.org, Dec 15 2016

Issue description

https://wicg.github.io/ResizeObserver/#resize-observer-entry-interface

Blink's ResizeObserverEntry.idl does not have a constructor.
 

Comment 1 by atotic@chromium.org, Dec 16 2016

Status: WontFix (was: Untriaged)
Code is correct, should fix the spec:

https://github.com/WICG/ResizeObserver/issues/35

Comment 2 by atotic@chromium.org, Dec 28 2016

@foolip

My code and spec format are similar to IntersectionObserver.

idl files:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/observer/ResizeObserverEntry.idl
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/dom/IntersectionObserverEntry.idl

- both do not define constructor in implementation idl

specs:
https://wicg.github.io/IntersectionObserver/#intersection-observer-entry
https://wicg.github.io/ResizeObserver/#resize-observer-entry-interface

- both define constructor in spec idl
- only ResizeObserver specifies constructor implementation.

What do you suggest I fix?



Status: ExternalDependency (was: WontFix)
IntersectionObserverEntry is issue 674519, so filing a spec issue for that too would be a good idea. If there's no particularly good reason to have the constructors, just removing them from the spec seems fine to me.

Reopening to keep track of this until spec and implementation agree.
Components: Blink>Layout
Cc: -atotic@chromium.org
Owner: atotic@chromium.org
Status: Assigned (was: ExternalDependency)

Sign in to add a comment