Summary: | DOMWindows should register themselves with the GamepadManager when they have event listeners | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brady Eidson <beidson> | ||||||
Component: | WebCore Misc. | Assignee: | Brady Eidson <beidson> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, dino, esprehn+autocc, kangil.han | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 134386 | ||||||||
Attachments: |
|
Description
Brady Eidson
2014-07-06 17:08:59 PDT
Created attachment 234471 [details]
Patch v1
This patch relies on the patch in 134670, so it probably won't apply right now. I'm just attaching it for posterity.
Created attachment 234538 [details]
Patch v1a (rebased, ready to EWS)
Attachment 234538 [details] did not pass style-queue:
ERROR: Source/WebCore/dom/EventNames.h:285: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
ERROR: Source/WebCore/dom/EventNames.h:288: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 2 in 6 files
If any of these errors are false positives, please file a bug against check-webkit-style.
(In reply to comment #3) > Attachment 234538 [details] did not pass style-queue: > > > ERROR: Source/WebCore/dom/EventNames.h:285: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] > ERROR: Source/WebCore/dom/EventNames.h:288: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] > Total errors found: 2 in 6 files > > > If any of these errors are false positives, please file a bug against check-webkit-style. While these macro newlines and comments are non-standard style, they're quite consistent within this file. =/ Comment on attachment 234538 [details] Patch v1a (rebased, ready to EWS) Clearing flags on attachment: 234538 Committed r170911: <http://trac.webkit.org/changeset/170911> All reviewed patches have been landed. Closing bug. |