RESOLVED FIXED160700
Implement "dumb" WK2 support for gamepads 
https://bugs.webkit.org/show_bug.cgi?id=160700
Summary Implement "dumb" WK2 support for gamepads
Brady Eidson
Reported 2016-08-09 11:20:08 PDT
Implement "dumb" WK2 support for gamepads The support is "dumb" because: - It uses a timer to refresh gamepad values instead of syncing to the display as spec'ed - It sends gamepad updates to all tabs instead of just the focused tab Those are tracked by https://bugs.webkit.org/show_bug.cgi?id=160673 and https://bugs.webkit.org/show_bug.cgi?id=160699, respectively.
Attachments
Patch (39.90 KB, patch)
2016-08-09 11:32 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (40.50 KB, patch)
2016-08-09 12:05 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2016-08-09 11:32:00 PDT
Created attachment 285652 [details] Patch
Brady Eidson
Comment 2 2016-08-09 12:05:05 PDT
Created attachment 285658 [details] Patch
Alex Christensen
Comment 3 2016-08-09 12:10:15 PDT
Comment on attachment 285658 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=285658&action=review r=me. Verified functionality manually. We really need some tests. > Source/WebKit2/Shared/Gamepad/GamepadData.h:50 > + String loggingString() const; #if !LOG_DISABLED around this?
WebKit Commit Bot
Comment 4 2016-08-09 14:30:33 PDT
Comment on attachment 285658 [details] Patch Clearing flags on attachment: 285658 Committed r204297: <http://trac.webkit.org/changeset/204297>
WebKit Commit Bot
Comment 5 2016-08-09 14:30:37 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.