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

Unified Diff: chrome/browser/android/vr_shell/vr_shell.cc

Issue 2879973002: Expose Gamepad API instance for Cardboard button (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/vr_shell/vr_shell.cc
diff --git a/chrome/browser/android/vr_shell/vr_shell.cc b/chrome/browser/android/vr_shell/vr_shell.cc
index 5d7b6bbb6ea9851693c9ed9fecc6710b1cadaaee..e8a748ee9ca19d38a1e51d20254b7dbbc933e37e 100644
--- a/chrome/browser/android/vr_shell/vr_shell.cc
+++ b/chrome/browser/android/vr_shell/vr_shell.cc
@@ -224,10 +224,12 @@ void VrShell::OnContentPaused(bool paused) {
delegate_provider_->device_provider()->Device()->OnFocus();
}
-void VrShell::OnTriggerEvent(JNIEnv* env, const JavaParamRef<jobject>& obj) {
+void VrShell::OnTriggerEvent(JNIEnv* env,
+ const JavaParamRef<jobject>& obj,
+ bool touched) {
gl_thread_->task_runner()->PostTask(
- FROM_HERE,
- base::Bind(&VrShellGl::OnTriggerEvent, gl_thread_->GetVrShellGl()));
+ FROM_HERE, base::Bind(&VrShellGl::OnTriggerEvent,
+ gl_thread_->GetVrShellGl(), touched));
}
void VrShell::OnPause(JNIEnv* env, const JavaParamRef<jobject>& obj) {
@@ -533,19 +535,24 @@ void VrShell::UpdateGamepadData(device::GvrGamepadData pad) {
unsigned int device_id =
delegate_provider_->device_provider()->Device()->id();
+
+ // Two gamepad factories: one for screen only, and one for controller.
+ device::GamepadDataFetcherManager::GetInstance()->AddFactory(
+ new device::GvrGamepadDataFetcher::Factory(this, device_id, true));
+
device::GamepadDataFetcherManager::GetInstance()->AddFactory(
- new device::GvrGamepadDataFetcher::Factory(this, device_id));
+ new device::GvrGamepadDataFetcher::Factory(this, device_id, false));
gamepad_source_active_ = true;
}
- if (gamepad_data_fetcher_) {
- gamepad_data_fetcher_->SetGamepadData(pad);
+ for (auto* fetcher : gamepad_data_fetchers_) {
+ fetcher->SetGamepadData(pad);
}
}
void VrShell::RegisterGamepadDataFetcher(
device::GvrGamepadDataFetcher* fetcher) {
DVLOG(1) << __FUNCTION__ << "(" << fetcher << ")";
- gamepad_data_fetcher_ = fetcher;
+ gamepad_data_fetchers_.push_back(fetcher);
}
// ----------------------------------------------------------------------------

Powered by Google App Engine
This is Rietveld 408576698