Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(224)

Issue 8345027: Big patch to implement Chromium-side of Gamepad support (Closed)

Created:
9 years, 2 months ago by scottmg
Modified:
9 years ago
Reviewers:
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, dpranke-watch+content_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Big patch to implement Chromium-side of Gamepad support Relies on patches in WebKit https://bugs.webkit.org/show_bug.cgi?id=69451 https://bugs.webkit.org/show_bug.cgi?id=71518 https://bugs.webkit.org/show_bug.cgi?id=71753 BUG=79050 TEST=

Patch Set 1 #

Patch Set 2 : pull in some changes from chrome only branch #

Patch Set 3 : merge with ToT #

Patch Set 4 : comment/whitespace fixes #

Patch Set 5 : merge out stuff committed in other patches #

Patch Set 6 : improved shared memory synchronization, and use new deviceschanged notif #

Patch Set 7 : fix merge #

Patch Set 8 : fix shutdown in provider, add unittest #

Patch Set 9 : merge out a bunch more stuff that's landed #

Patch Set 10 : move gamepad_util out of webkit/glue #

Patch Set 11 : fix merge #

Patch Set 12 : try again? #

Patch Set 13 : remove unused instance var #

Patch Set 14 : merge out some more #

Patch Set 15 : merge out some more #

Patch Set 16 : update with split out of synch primitives for reader, and gamepad service to own browser thread #

Patch Set 17 : remove files #

Patch Set 18 : merge out more landed code #

Patch Set 19 : minor tidying #

Unified diffs Side-by-side diffs Delta from patch set Stats (+243 lines, -50 lines) Patch
M content/browser/gamepad/gamepad_provider.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 4 chunks +10 lines, -4 lines 0 comments Download
M content/browser/gamepad/gamepad_provider.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 6 chunks +41 lines, -23 lines 0 comments Download
M content/browser/gamepad/gamepad_provider_unittest.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 3 chunks +0 lines, -12 lines 0 comments Download
A content/browser/gamepad/gamepad_service.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 1 chunk +73 lines, -0 lines 0 comments Download
A content/browser/gamepad/gamepad_service.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 1 chunk +89 lines, -0 lines 0 comments Download
M content/browser/renderer_host/gamepad_browser_message_filter.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 2 chunks +12 lines, -2 lines 0 comments Download
M content/browser/renderer_host/gamepad_browser_message_filter.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 2 chunks +15 lines, -8 lines 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 1 chunk +1 line, -1 line 0 comments Download
M content/content_browser.gypi View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
scottmg
9 years ago (2011-12-02 18:27:39 UTC) #1
All code landed in other patches, closing now. See http://crbug.com/79050 for
tracking.

Powered by Google App Engine
This is Rietveld 408576698