Delay button bounds offset calculation until window is actually visible

chrome/browser/ui/views/frame/minimize_button_metrics_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/frame/minimize_button_metrics_win.h"
 
 #include "base/logging.h"
 #include "base/i18n/rtl.h"
 #include "base/win/windows_version.h"
 #include "dwmapi.h"
@@ skipping 55 matching lines @@
 // This function attempts to calculate the odd and varying difference
 // between the results of DwmGetWindowAttribute with the
 // DWMWA_CAPTION_BUTTON_BOUNDS flag and the information from the
 // WM_GETTITLEBARINFOEX message. It will return an empirically determined
 // offset until the window has been activated and the message returns
 // valid rectangles.
 int MinimizeButtonMetrics::GetButtonBoundsPositionOffset(
     const RECT& button_bounds,
     const RECT& window_bounds) const {
   if (button_bounds_position_offset_ == kInvalidOffset) {
-    if (!was_activated_)
+    if (!was_activated_ || !IsWindowVisible(hwnd_))
       return GetDefaultButtonBoundsOffset();
     TITLEBARINFOEX info = {0};
     info.cbSize = sizeof(info);
     SendMessage(hwnd_, WM_GETTITLEBARINFOEX, 0,
                 reinterpret_cast<LPARAM>(&info));
     if (info.rgrect[2].right == info.rgrect[2].left ||
         (info.rgstate[2] & (STATE_SYSTEM_INVISIBLE | STATE_SYSTEM_OFFSCREEN |
                             STATE_SYSTEM_UNAVAILABLE)))
       return GetDefaultButtonBoundsOffset();
     button_bounds_position_offset_ =
@@ skipping 95 matching lines @@
     cached_minimize_button_x_delta_ = minimize_button_offset;
   } else {
     RECT client_rect = {0};
     GetClientRect(hwnd_, &client_rect);
     cached_minimize_button_x_delta_ =
         client_rect.right - minimize_button_offset;
   }
   last_cached_minimize_button_x_delta_ = cached_minimize_button_x_delta_;
   return minimize_button_offset;
 }