From dde2fd6f3321c57be1534f1e1ef7ad1809924836 Mon Sep 17 00:00:00 2001
From: Michael Kirk <michael@signal.org>
Date: Thu, 12 Jul 2018 13:05:24 -0600
Subject: [PATCH] Hide menu window when vc dismisses.

Normally this isn't possible, just being paranoid.

// FREEBIE
---
 .../ConversationView/ConversationViewController.m            | 5 ++---
 SignalMessaging/utils/OWSWindowManager.m                     | 2 ++
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/Signal/src/ViewControllers/ConversationView/ConversationViewController.m b/Signal/src/ViewControllers/ConversationView/ConversationViewController.m
index 454666edb..9b9fb12ba 100644
--- a/Signal/src/ViewControllers/ConversationView/ConversationViewController.m
+++ b/Signal/src/ViewControllers/ConversationView/ConversationViewController.m
@@ -1206,6 +1206,8 @@ typedef enum : NSUInteger {
     [super viewWillDisappear:animated];
 
     self.isViewCompletelyAppeared = NO;
+
+    [[OWSWindowManager sharedManager] hideMenuActionsWindow];
 }
 
 - (void)viewDidDisappear:(BOOL)animated
@@ -2558,9 +2560,6 @@ typedef enum : NSUInteger {
     [self dismissViewControllerAnimated:NO completion:nil];
 }
 
-// TODO override dismissViewController to dismiss any presented MessagesActionSheet?
-// e.g. if we have any remaining dismiss-and-present logic remaining?
-
 #pragma mark - CNContactViewControllerDelegate
 
 - (void)contactViewController:(CNContactViewController *)viewController
diff --git a/SignalMessaging/utils/OWSWindowManager.m b/SignalMessaging/utils/OWSWindowManager.m
index dd35b9cb1..1d128890d 100644
--- a/SignalMessaging/utils/OWSWindowManager.m
+++ b/SignalMessaging/utils/OWSWindowManager.m
@@ -269,6 +269,8 @@ const UIWindowLevel UIWindowLevel_MessageActions(void)
 
 - (void)showMenuActionsWindow:(UIViewController *)menuActionsViewController
 {
+    OWSAssert(self.menuActionsViewController == nil);
+
     self.menuActionsViewController = menuActionsViewController;
     self.menuActionsWindow.rootViewController = menuActionsViewController;