Merge pull request #476 from adobe/jeff/fix-9597

Don't Center Popups if Size and Width aren't parsed
adobe · Oct 20, 2014 · e9cd327 · e9cd327
2 parents 330764d + be27fe7
commit e9cd327
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/appshell/client_handler_win.cpp b/appshell/client_handler_win.cpp
@@ -123,6 +123,17 @@ bool ClientHandler::OnKeyEvent(CefRefPtr<CefBrowser> browser,
 
  void ClientHandler::ComputePopupPlacement(CefWindowInfo& windowInfo)
  {
+      // both must be set to work
+      if (windowInfo.width == CW_USEDEFAULT ||
+          windowInfo.height == CW_USEDEFAULT) {
+            // force both vals to be CW_USEDEFAULT in this
+            //  case because Windows doesn't correctly handle 
+            //  only one value being is supplied on input
+            windowInfo.width = CW_USEDEFAULT;
+            windowInfo.height = CW_USEDEFAULT;
+         return;
+     }
+
      RECT rectMainWnd;
      gMainWnd->GetWindowRect(&rectMainWnd);