Views: Fixes BrowserNonClientFrameViewMac Layout trigger

This CL updates the non client frame view to invalidate its layout
in fullscreen only if the frame is meant to be shown.

This eliminates unexpected behavior incurred when laying out the
frame view when it should not be shown.

Bug: 1119652
Change-Id: I702f4c4ab913e654b03b4327e60d26ef60831446
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2393002Reviewed-by: Allen Bauer <kylixrd@chromium.org>
Commit-Queue: Thomas Lukaszewicz <tluk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#804427}

chrome/browser/ui/views/frame/browser_non_client_frame_view_mac.mm

@@ -217,8 +217,8 @@ void BrowserNonClientFrameViewMac::UpdateFullscreenTopUI() {
   if (frame()->IsFullscreen()) {
     browser_view()->Layout();
     // The web frame toolbar is visible in fullscreen mode on Mac and thus
-    // requires a re-layout when in fullscreen.
-    if (web_app_frame_toolbar())
+    // requires a re-layout when in fullscreen and shown.
+    if (web_app_frame_toolbar() && !ShouldHideTopUIForFullscreen())
       InvalidateLayout();
   }
 }