Quit application when SteamVR requests it

interface/src/Application.cpp

@@ -5144,13 +5144,6 @@ void Application::updateDisplayMode() {
             QObject::connect(displayPlugin.get(), &DisplayPlugin::recommendedFramebufferSizeChanged, [this](const QSize & size) {
                 resizeGL();
             });
-            QObject::connect(displayPlugin.get(), &DisplayPlugin::outputDeviceLost, [this, displayPluginName] {
-                PluginManager::getInstance()->disableDisplayPlugin(displayPluginName);
-                auto menu = Menu::getInstance();
-                if (menu->menuItemExists(MenuOption::OutputMenu, displayPluginName)) {
-                    menu->removeMenuItem(MenuOption::OutputMenu, displayPluginName);
-                }
-            });
             first = false;
         }
 

libraries/plugins/src/plugins/DisplayPlugin.h

@@ -170,10 +170,6 @@ public:
 
 signals:
     void recommendedFramebufferSizeChanged(const QSize & size);
-    // Indicates that this display plugin is no longer valid for use.
-    // For instance if a user exits Oculus Home or Steam VR while 
-    // using the corresponding plugin, that plugin should be disabled.
-    void outputDeviceLost();
 
 protected:
     void incrementPresentCount();

plugins/openvr/src/OpenVrDisplayPlugin.cpp

@@ -128,10 +128,7 @@ void OpenVrDisplayPlugin::resetSensors() {
 
 bool OpenVrDisplayPlugin::beginFrameRender(uint32_t frameIndex) {
     handleOpenVrEvents();
-    if (openVrQuitRequested()) {
-        emit outputDeviceLost();
-        return false;
-    }
+
     double displayFrequency = _system->GetFloatTrackedDeviceProperty(vr::k_unTrackedDeviceIndex_Hmd, vr::Prop_DisplayFrequency_Float);
     double frameDuration = 1.f / displayFrequency;
     double vsyncToPhotons = _system->GetFloatTrackedDeviceProperty(vr::k_unTrackedDeviceIndex_Hmd, vr::Prop_SecondsFromVsyncToPhotons_Float);

plugins/openvr/src/OpenVrHelpers.cpp

@@ -26,11 +26,6 @@ using Lock = std::unique_lock<Mutex>;
 static int refCount { 0 };
 static Mutex mutex;
 static vr::IVRSystem* activeHmd { nullptr };
-static bool _openVrQuitRequested { false };
-
-bool openVrQuitRequested() {
-    return _openVrQuitRequested;
-}
 
 static const uint32_t RELEASE_OPENVR_HMD_DELAY_MS = 5000;
 
@@ -84,7 +79,6 @@ void releaseOpenVrSystem() {
         if (0 == refCount) {
             qCDebug(displayplugins) << "OpenVR: zero refcount, deallocate VR system";
             vr::VR_Shutdown();
-            _openVrQuitRequested = false;
             activeHmd = nullptr;
         }
     }
@@ -103,14 +97,14 @@ void handleOpenVrEvents() {
     while (activeHmd->PollNextEvent(&event, sizeof(event))) {
         switch (event.eventType) {
             case vr::VREvent_Quit:
-                _openVrQuitRequested = true;
                 activeHmd->AcknowledgeQuit_Exiting();
+                QMetaObject::invokeMethod(qApp, "quit");
                 break;
 
             default:
                 break;
         }
-        qDebug() << "OpenVR: Event " << event.eventType;
+        qDebug() << "OpenVR: Event " << activeHmd->GetEventTypeNameFromEnum((vr::EVREventType)event.eventType) << "(" << event.eventType << ")";
     }
 
 }

plugins/openvr/src/ViveControllerManager.cpp

@@ -214,10 +214,6 @@ void ViveControllerManager::renderHand(const controller::Pose& pose, gpu::Batch&
 void ViveControllerManager::pluginUpdate(float deltaTime, const controller::InputCalibrationData& inputCalibrationData) {
     auto userInputMapper = DependencyManager::get<controller::UserInputMapper>();
     handleOpenVrEvents();
-    if (openVrQuitRequested()) {
-        deactivate();
-        return;
-    }
 
     // because update mutates the internal state we need to lock
     userInputMapper->withLock([&, this]() {