Make sure that the window is repositioned and if applicable visible before reparenting it.

This is to workaround a strange behavior in Windows, where if the parent window is from
a different thread and the window becomes visible after reparenting it, the thread inputs
are not attached correctly causing focus and other issues.


Review URL: http://codereview.chromium.org/67292

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14025 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 3 deletions

diff --git a/chrome/browser/automation/automation_provider.cc b/chrome/browser/automation/automation_provider.cc
index bac531e5..ba4dfe4 100644
--- a/chrome/browser/automation/automation_provider.cc
+++ b/chrome/browser/automation/automation_provider.cc
@@ -2837,15 +2837,15 @@ void AutomationProvider::OnTabReposition(
     return;
   }
 
+  SetWindowPos(params.window, params.window_insert_after, params.left,
+               params.top, params.width, params.height, params.flags);
+
   if (params.set_parent) {
     if (IsWindow(params.parent_window)) {
       if (!SetParent(params.window, params.parent_window))
         DLOG(WARNING) << "SetParent failed. Error 0x%x" << GetLastError();
     }
   }
-
-  SetWindowPos(params.window, params.window_insert_after, params.left,
-               params.top, params.width, params.height, params.flags);
 }
 
 #endif  // defined(OS_WIN)