New issue
Advanced search Search tips

Issue 730030 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Exp-Leadership: ----
Launch-Leadership: ----
Launch-Legal: ----
Launch-M-Approved: 61-Dev , 61-Beta , 61-Stable
Launch-M-Target: 61-Dev , 61-Beta , 61-Stable
Launch-Privacy: ----
Launch-Security: ----
Launch-Test: ----
Launch-UI: ----
Rollout-Type: ----



Sign in to add a comment

onwheel is not on the appropriate location

Project Member Reported by domenic@chromium.org, Jun 6 2017

Issue description

Chrome Version: 58.0.3029.110 (64-bit)

See tests at https://github.com/w3c/web-platform-tests/pull/6165. Unlike all other (implemented) GlobalEventHandlers, onwheel is not on HTMLElement.

 
Cc: dtapu...@chromium.org nzolghadr@chromium.org
Labels: Hotlist-Input-Dev
Owner: eirage@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/033440cd8501cdc13857fe8b6b317faa62d7be03

commit 033440cd8501cdc13857fe8b6b317faa62d7be03
Author: eirage <eirage@chromium.org>
Date: Mon Jul 10 16:12:48 2017

Move onwheel from Element.idl to GlobalEventHandlers

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/Wa5sdcDrJnA

BUG= 730030 

Review-Url: https://codereview.chromium.org/2936263003
Cr-Commit-Position: refs/heads/master@{#485285}

[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/external/wpt/html/dom/interfaces-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/external/wpt/html/webappapis/scripting/events/event-handler-all-global-events-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/fast/dom/global-event-handlers-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/fast/dom/global-event-handlers.html
[add] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/platform/mac/virtual/stable/webexposed/OWNERS
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/platform/mac/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/platform/win/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/virtual/service-worker-navigation-preload-disabled/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/dom/Document.h
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/dom/Document.idl
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/dom/Element.h
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/dom/Element.idl
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/dom/GlobalEventHandlers.h
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/dom/GlobalEventHandlers.idl
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/frame/LocalDOMWindow.h
[modify] https://crrev.com/033440cd8501cdc13857fe8b6b317faa62d7be03/third_party/WebKit/Source/core/frame/Window.idl

Comment 3 by eirage@chromium.org, Jul 10 2017

Status: Fixed (was: Assigned)
Labels: -Type-Bug M-61 Type-Launch-OWP
Labels: Launch-M-Approved-61-Dev Launch-M-Approved-61-Beta Launch-M-Approved-61-Stable Launch-M-Target-61-Dev Launch-M-Target-61-Beta Launch-M-Target-61-Stable

Sign in to add a comment