Fix DirectManipulationBrowserTest.HWNDReparent in Debug bots

The test failed in debug bots because we direct reparent the HWND to
another shell and skip full window reparent logic then shell close also
overwrite some state.

In this patch, we moved shell close to the end of test body.

Bug: 833661
Change-Id: I06c639a1c1620fe3a06af2eb6606970a6a86c78c
Reviewed-on: https://chromium-review.googlesource.com/1025119Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Jianpeng Chao <chaopeng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553259}

content/browser/renderer_host/direct_manipulation_browsertest.cc

@@ -113,10 +113,10 @@ IN_PROC_BROWSER_TEST_P(DirectManipulationBrowserTest, HWNDReparent) {
   UpdateParent(
       shell2->window()->GetRootWindow()->GetHost()->GetAcceleratedWidget());
 
-  shell()->Close();
-
   // The animation observer should be removed.
   EXPECT_FALSE(HasCompositorAnimationObserver(lrwhh));
+
+  shell2->Close();
 }
 
 // EventLogger is to obserser the events sent from WindowEventTarget (the root