From f90a351400c5de2a89a30c25dc72a00fe25ca8ab Mon Sep 17 00:00:00 2001
From: Brad Davis <bdavis@saintandreas.org>
Date: Fri, 24 Jun 2016 16:31:24 -0700
Subject: [PATCH 1/2] Don't reposition windows prior to selecting the initial
 display plugin

---
 interface/resources/qml/desktop/Desktop.qml | 16 +++++++++++++++-
 interface/src/Application.cpp               |  8 +++++++-
 2 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/interface/resources/qml/desktop/Desktop.qml b/interface/resources/qml/desktop/Desktop.qml
index 9f10cfc64a..27fa9692b9 100644
--- a/interface/resources/qml/desktop/Desktop.qml
+++ b/interface/resources/qml/desktop/Desktop.qml
@@ -24,6 +24,13 @@ FocusScope {
     readonly property int invalid_position: -9999;
     property rect recommendedRect: Qt.rect(0,0,0,0);
     property var expectedChildren;
+    property bool repositionLocked: true
+
+    onRepositionLockedChanged: {
+        if (!repositionLocked) {
+            d.handleSizeChanged();
+        }
+    }
 
     onHeightChanged: d.handleSizeChanged();
     
@@ -52,11 +59,14 @@ FocusScope {
         readonly property real menu: 8000
     }
 
-
     QtObject {
         id: d
 
         function handleSizeChanged() {
+            if (desktop.repositionLocked) {
+                return;
+            }
+
             var oldRecommendedRect = recommendedRect;
             var newRecommendedRectJS = (typeof Controller === "undefined") ? Qt.rect(0,0,0,0) : Controller.getRecommendedOverlayRect();
             var newRecommendedRect = Qt.rect(newRecommendedRectJS.x, newRecommendedRectJS.y, 
@@ -235,6 +245,10 @@ FocusScope {
         }
 
         function repositionAll() {
+            if (desktop.repositionLocked) {
+                return;
+            }
+
             var oldRecommendedRect = recommendedRect;
             var oldRecommendedDimmensions = { x: oldRecommendedRect.width, y: oldRecommendedRect.height };
             var newRecommendedRect = Controller.getRecommendedOverlayRect();
diff --git a/interface/src/Application.cpp b/interface/src/Application.cpp
index 26da43fb58..858b86025f 100644
--- a/interface/src/Application.cpp
+++ b/interface/src/Application.cpp
@@ -963,6 +963,13 @@ Application::Application(int& argc, char** argv, QElapsedTimer& startupTimer) :
     updateHeartbeat();
 
     loadSettings();
+
+    // Now that we've loaded the menu and thus switched to the previous display plugin
+    // we can unlock the desktop repositioning code, since all the positions will be 
+    // relative to the desktop size for this plugin
+    auto offscreenUi = DependencyManager::get<OffscreenUi>();
+    offscreenUi->getDesktop()->setProperty("repositionLocked", false);
+
     // Make sure we don't time out during slow operations at startup
     updateHeartbeat();
 
@@ -5348,7 +5355,6 @@ void Application::updateDisplayMode() {
         _displayPlugin = newDisplayPlugin;
     }
 
-
     emit activeDisplayPluginChanged();
 
     // reset the avatar, to set head and hand palms back to a reasonable default pose.

From 68be819ea25eecc159d1034869098a2f8e125b91 Mon Sep 17 00:00:00 2001
From: howard-stearns <howard.stearns@gmail.com>
Date: Mon, 27 Jun 2016 14:07:13 -0700
Subject: [PATCH 2/2] fix crash when hmd context menu triggers a modal

---
 .../resources/qml/menus/MenuMouseHandler.qml     | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/interface/resources/qml/menus/MenuMouseHandler.qml b/interface/resources/qml/menus/MenuMouseHandler.qml
index 9ba158cb28..48574d41e5 100644
--- a/interface/resources/qml/menus/MenuMouseHandler.qml
+++ b/interface/resources/qml/menus/MenuMouseHandler.qml
@@ -39,6 +39,19 @@ Item {
                 onSelected: d.handleSelection(subMenu, currentItem, item)
             }
         }
+        property var delay: Timer { // No setTimeout in QML.
+            property var menuItem: null;
+            interval: 0
+            repeat: false
+            running: false
+            function trigger(item) { // Capture item and schedule asynchronous Timer.
+                menuItem = item;
+                start();
+            }
+            onTriggered: {
+                menuItem.trigger(); // Now trigger the item.
+            }
+        }
 
         function toModel(items) {
             var result = modelMaker.createObject(desktop);
@@ -128,7 +141,8 @@ Item {
 
                 case MenuItemType.Item:
                     console.log("Triggering " + item.text)
-                    item.trigger();
+                    // Don't block waiting for modal dialogs and such that the menu might open.
+                    delay.trigger(item);
                     clearMenus();
                     break;
                 }