Menu is now properly hidden when toggled (only used in Brave for Linux).

Initial PR was done with #191 but I missed electron/electron@dcf6c52 Fixes brave/browser-laptop#8839 Auditors: @bbondy, @bridiver
brave · May 17, 2017 · 4b04c7b · 4b04c7b
1 parent 89405c8
commit 4b04c7b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/atom/browser/native_window_views.cc b/atom/browser/native_window_views.cc
@@ -1277,7 +1277,8 @@ void NativeWindowViews::HandleKeyboardEvent(
 void NativeWindowViews::Layout() {
   const auto size = GetContentsBounds().size();
   const auto menu_bar_bounds =
-      menu_bar_ ? gfx::Rect(0, 0, size.width(), kMenuBarHeight) : gfx::Rect();
+      menu_bar_visible_ ? gfx::Rect(0, 0, size.width(), kMenuBarHeight)
+                        : gfx::Rect();
   if (menu_bar_) {
     menu_bar_->SetBoundsRect(menu_bar_bounds);
   }