(js) Improve popup window detection in mailer

Fixes #4518

NEWS

@@ -9,6 +9,7 @@ Enhancements
 
 Bug fixes
  - [core] handle multi-valued mozillasecondemail attribute mapping
+ - [web] improve popup window detection in message viewer (#4518)
 
 4.0.1 (2018-07-10)
 ------------------

UI/WebServerResources/js/Mailer/MessageController.js

@@ -11,6 +11,8 @@
     var vm = this, popupWindow = null, hotkeys = [];
 
     this.$onInit = function() {
+      var isPopupWindow = false;
+
       // Expose controller
       $window.$messageController = vm;
 
@@ -30,9 +32,14 @@
 
       _registerHotkeys(hotkeys);
 
+      // Detect if this is message appears in a separate window
+      try {
+        isPopupWindow = $window.opener && '$mailboxController' in $window.opener;
+      }
+      catch (e) {}
+
       // One-way refresh of the parent window when modifying the message from a popup window.
-      if ($window.opener &&
-          '$mailboxController' in $window.opener) {
+      if (isPopupWindow) {
         // Update the message flags. The message must be displayed in the parent window.
         $scope.$watchCollection(function() { return vm.message.flags; }, function(newTags, oldTags) {
           var ctrls;