Bug 1412509 - Follow bug 1411642 with a change in msgMail3PaneWindow.js. r=jorgk CLOSED TREE
authorRichard Marti <richard.marti@gmail.com>
Sat, 28 Oct 2017 13:01:31 +0200
changeset 29280 891c8dc1e07ed89c8e5a3187f2649e32f57561ae
parent 29279 59043b5664802c8d42540fe352eb40446b6fcbe3
child 29281 bf29aafeeae317b121c7e1f9a711967f22a14d48
push id2068
push userclokep@gmail.com
push dateMon, 13 Nov 2017 19:02:14 +0000
treeherdercomm-beta@9c7e7ce8672b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjorgk
bugs1412509, 1411642
Bug 1412509 - Follow bug 1411642 with a change in msgMail3PaneWindow.js. r=jorgk CLOSED TREE
mail/base/content/msgMail3PaneWindow.js
--- a/mail/base/content/msgMail3PaneWindow.js
+++ b/mail/base/content/msgMail3PaneWindow.js
@@ -1832,21 +1832,20 @@ var TabsInTitlebar = {
 
       // And get the height of what's in the titlebar:
       let titlebarContentHeight = rect(titlebarContent).height;
 
       // Begin setting CSS properties which will cause a reflow
 
       // On Windows 10, adjust the window controls to span the entire
       // tab strip height if we're not showing a menu bar.
-      if (AppConstants.isPlatformAndVersionAtLeast("win", "10.0")) {
-        if (!menuHeight) {
-          titlebarContentHeight = fullTabsHeight;
-          $("titlebar-buttonbox").style.height = titlebarContentHeight + "px";
-        }
+      if (AppConstants.isPlatformAndVersionAtLeast("win", "10.0") &&
+          !menuHeight) {
+        titlebarContentHeight = fullTabsHeight;
+        $("titlebar-buttonbox").style.height = titlebarContentHeight + "px";
       }
 
       // If the menubar is around (menuHeight is non-zero), try to adjust
       // its full height (i.e. including margins) to match the titlebar,
       // by changing the menubar's bottom padding
       if (menuHeight) {
         // Calculate the difference between the titlebar's height and that of
         // the menubar