WK2 Gamepad provider on iOS
Created attachment 287448 [details] Patch
Comment on attachment 287448 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=287448&action=review > Source/WebKit2/Platform/spi/ios/UIKitSPI.h:829 > +- (UIResponder *) _responderForKeyEvents; no space after the ) > Source/WebKit2/UIProcess/Gamepad/ios/UIGamepadProviderIOS.mm:38 > + auto firstResponder = [[UIApplication sharedApplication] _responderForKeyEvents]; use firstResponder, I think
Created attachment 287451 [details] Patch
Created attachment 287461 [details] Patch
Created attachment 287483 [details] Patch
Created attachment 287485 [details] Patch
Comment on attachment 287485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=287485&action=review > Source/WebCore/platform/gamepad/cocoa/GameControllerGamepad.mm:28 > +#if ENABLE(GAMEPAD) && (defined(__LP64__) || PLATFORM(IOS)) I think we should have a USE(GAME_CONTROLLER). This is quite unintuitive.
Comment on attachment 287485 [details] Patch Clearing flags on attachment: 287485 Committed r205247: <http://trac.webkit.org/changeset/205247>
All reviewed patches have been landed. Closing bug.
(In reply to comment #7) > Comment on attachment 287485 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=287485&action=review > > > Source/WebCore/platform/gamepad/cocoa/GameControllerGamepad.mm:28 > > +#if ENABLE(GAMEPAD) && (defined(__LP64__) || PLATFORM(IOS)) > > I think we should have a USE(GAME_CONTROLLER). This is quite unintuitive. Since its spread out all over at this point, will do in a follow up