New issue
Advanced search Search tips

Issue 663633 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Clean up Event related class's constructor's parameter

Project Member Reported by hayato@chromium.org, Nov 9 2016

Issue description

There are a lot of overloaded constructors in Event related classes. Each of them has many parameters.

It is difficult to know which constructor (via XXXEvent::create() factory) should be used.
I would feel that it is very error-prone and difficult to maintain.

For example, a double click event is being wrongly created without `composed` flag being set.
https://bugs.chromium.org/p/chromium/issues/detail?id=662401

It would be better to clean up these constructors somehow to prevent further potential mistakes.
 
Description: Show this description

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

Components: -Blink>DOM>Events Blink>DOM
Remove Blink>DOM>Events
Labels: Hotlist-GoodFirstBug
Owner: ----
Status: Available (was: Assigned)

Sign in to add a comment