Standardize or remove non-standard parts of Blink's IDL |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Issue descriptionBlink's IDL files have many interfaces, attributes, methods, etc. that are not in any spec. Many have never been considered for standardization, while other were once in a spec but were then removed. For each of these APIs, we should aim to either standardize it and see it implemented in all engines, or removed from Blink and any other engine that implement it. Advise on how to tackle these issues is in this "Finding a path to interop" document: https://docs.google.com/document/d/1LSuLWJDP02rlC9bOlidL6DzBV5kSkV5bW5Pled8HGC8/edit?usp=sharing ⛆ |
|
|
Showing comments 146 - 245
of 245
Older ›
,
Mar 9 2017
,
Mar 10 2017
,
Mar 10 2017
,
Mar 10 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 15 2017
,
Mar 15 2017
,
Mar 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/510e3b3a9e0d1fb0a132d9f3c2e762e6516b0bb8 commit 510e3b3a9e0d1fb0a132d9f3c2e762e6516b0bb8 Author: lunalu <lunalu@chromium.org> Date: Thu Mar 16 20:08:32 2017 Added spec links to CSSTransformComponent and StylePropertyMap BUG=674593 Review-Url: https://codereview.chromium.org/2744713004 Cr-Commit-Position: refs/heads/master@{#457531} [modify] https://crrev.com/510e3b3a9e0d1fb0a132d9f3c2e762e6516b0bb8/third_party/WebKit/Source/core/css/cssom/CSSTransformComponent.idl [modify] https://crrev.com/510e3b3a9e0d1fb0a132d9f3c2e762e6516b0bb8/third_party/WebKit/Source/core/css/cssom/StylePropertyMap.idl
,
Mar 20 2017
,
Mar 20 2017
,
Mar 20 2017
,
Mar 20 2017
,
Mar 21 2017
,
Mar 23 2017
,
Mar 29 2017
,
Mar 29 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Mar 31 2017
,
Apr 3 2017
,
Apr 3 2017
,
Apr 3 2017
,
Apr 4 2017
,
Apr 4 2017
,
Apr 4 2017
,
Apr 4 2017
,
Apr 4 2017
,
Apr 6 2017
,
Apr 6 2017
,
Apr 6 2017
,
Apr 6 2017
,
Apr 6 2017
,
Apr 6 2017
,
Apr 6 2017
I am attaching a CSV files of (open) the blocking bugs as of now. This is in order to later be able to tell how many have been resolved in Q2, or later.
,
Apr 12 2017
,
Apr 14 2017
,
Apr 14 2017
,
Apr 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ba3e08e72dbeacfc92d01648ac93a72a26257d36 commit ba3e08e72dbeacfc92d01648ac93a72a26257d36 Author: jsbell <jsbell@chromium.org> Date: Tue Apr 25 21:00:51 2017 IndexedDB: Add [SameObject] / [NewObject] annotations per spec A recent spec change [1] to IndexedDB [2] adds the [SameObject] [3] and [NewObject] [4] IDL extended attributes to properties and methods as appropiate. These don't change the behavior (which was described by algorithmic prose) but allow implementations/tests to make additional assertions about the behavior of the interfaces. [1] https://github.com/w3c/IndexedDB/commit/05b2a7c5d65a497c49c8636021fc1cc791fdc9c9 [2] https://w3c.github.io/IndexedDB/ [3] https://heycam.github.io/webidl/#SameObject [4] https://heycam.github.io/webidl/#NewObject BUG=674593 R=foolip@chromium.org Review-Url: https://codereview.chromium.org/2837613003 Cr-Commit-Position: refs/heads/master@{#467108} [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBCursor.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBDatabase.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBFactory.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBIndex.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBKeyRange.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBObjectStore.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBRequest.idl [modify] https://crrev.com/ba3e08e72dbeacfc92d01648ac93a72a26257d36/third_party/WebKit/Source/modules/indexeddb/IDBTransaction.idl
,
Apr 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0626510c113d95afa8aa15666a81bb25fc7fbda3 commit 0626510c113d95afa8aa15666a81bb25fc7fbda3 Author: jsbell <jsbell@chromium.org> Date: Wed Apr 26 00:57:51 2017 IndexedDB: Remove DOMStringList special case for transaction() Now that sequence conversion handles iterables, we no longer need a special case for IDBDatabase's transaction() method which takes a union of (DOMString or sequence<DOMString>) to accept DOMStringList. The method signature now matches the spec. BUG=674593 TEST=storage/indexeddb/transaction-basics.html Review-Url: https://codereview.chromium.org/2840893002 Cr-Commit-Position: refs/heads/master@{#467188} [modify] https://crrev.com/0626510c113d95afa8aa15666a81bb25fc7fbda3/third_party/WebKit/Source/bindings/modules/v8/generated.gni [modify] https://crrev.com/0626510c113d95afa8aa15666a81bb25fc7fbda3/third_party/WebKit/Source/modules/indexeddb/IDBDatabase.cpp [modify] https://crrev.com/0626510c113d95afa8aa15666a81bb25fc7fbda3/third_party/WebKit/Source/modules/indexeddb/IDBDatabase.h [modify] https://crrev.com/0626510c113d95afa8aa15666a81bb25fc7fbda3/third_party/WebKit/Source/modules/indexeddb/IDBDatabase.idl [modify] https://crrev.com/0626510c113d95afa8aa15666a81bb25fc7fbda3/third_party/WebKit/Source/modules/indexeddb/InspectorIndexedDBAgent.cpp
,
Apr 27 2017
,
May 25 2017
,
Jun 22 2017
,
Jul 4 2017
Attaching a new CSV files of the issues that are blocking today, for Q3 OKR scoring.
,
Sep 27 2017
,
Oct 4 2017
,
Nov 29 2017
,
Feb 26 2018
,
Mar 2 2018
,
Mar 3 2018
,
May 18 2018
,
Aug 30
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5e13925e9de78a42421c5a4e8c884ef9dea97849 commit 5e13925e9de78a42421c5a4e8c884ef9dea97849 Author: Hitoshi Yoshida <peria@chromium.org> Date: Thu Aug 30 04:30:26 2018 IDL: Move [Unforgeable] on Location interface to its members An IDL extended attribute `[Unforgeable]` is not applicable on interfaces, but we have it on Location interface. This CL removes it from interface definition and applies it on its members (operations and attributes) to follow Web IDL spec and HTML spec. Bug: 674593 Change-Id: I295d7c80a39304d7ec497c0492ca6061c153036a Reviewed-on: https://chromium-review.googlesource.com/1194528 Reviewed-by: Kentaro Hara <haraken@chromium.org> Reviewed-by: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by: Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#587442} [modify] https://crrev.com/5e13925e9de78a42421c5a4e8c884ef9dea97849/third_party/blink/renderer/core/frame/location.idl
,
Aug 31
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ee0ee3d48025f31776bc33cd38b36df0504a02f3 commit ee0ee3d48025f31776bc33cd38b36df0504a02f3 Author: Hitoshi Yoshida <peria@chromium.org> Date: Fri Aug 31 09:13:29 2018 IDL compiler: Disable to put [Unforgeable] on interfaces or static members In WebIDL spec, [Unforgeable] can appear on interface attributes and interface non-static operations. https://heycam.github.io/webidl/#Unforgeable Now our IDL compiler does not check its use cases. This CL makes IDL compiler to check if [Unforgeable] appears on interfaces or static members, and stops with a warning message if it does. Bug: 674593 Change-Id: I6f17834eae71b3c7aa44556c3442f679091efd59 Reviewed-on: https://chromium-review.googlesource.com/1198711 Reviewed-by: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by: Yuki Shiino <yukishiino@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#587973} [modify] https://crrev.com/ee0ee3d48025f31776bc33cd38b36df0504a02f3/third_party/blink/renderer/bindings/IDLExtendedAttributes.md [modify] https://crrev.com/ee0ee3d48025f31776bc33cd38b36df0504a02f3/third_party/blink/renderer/bindings/scripts/compute_interfaces_info_individual.py [modify] https://crrev.com/ee0ee3d48025f31776bc33cd38b36df0504a02f3/third_party/blink/renderer/bindings/scripts/idl_definitions.py [modify] https://crrev.com/ee0ee3d48025f31776bc33cd38b36df0504a02f3/third_party/blink/renderer/bindings/scripts/v8_attributes.py [modify] https://crrev.com/ee0ee3d48025f31776bc33cd38b36df0504a02f3/third_party/blink/renderer/bindings/scripts/v8_methods.py [modify] https://crrev.com/ee0ee3d48025f31776bc33cd38b36df0504a02f3/third_party/blink/renderer/bindings/scripts/v8_utilities.py
,
Sep 27
,
Oct 25
Showing comments 146 - 245
of 245
Older ›
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||