Makes NativeViewHostWin change the parent of the hwnd when

Makes NativeViewHostWin change the parent of the hwnd when
detaching. Without this the hwnd would remain parented so that if
the widget containing the NativeViewHost was destroyed then the
hwnd would get destroyed by virtue of being a child.

Previously we had this logic in NativeTabContentsViewWin (lost in
WebView refactoring), but it seems better to promote to
NativeViewHostWin.

BUG=132090
TEST=none
R=ben@chromium.org,jam@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149459

[jam, 2012-07-25 17:02:16]

lgtm

[Ben Goodger (Google), 2012-07-27 17:21:25]

I suck and that's sad, sorry for taking so long.

LGTM

[commit-bot: I haz the power, 2012-07-28 02:01:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/10832011/2001

[commit-bot: I haz the power, 2012-07-28 04:52:39]

Try job failure for 10832011-2001 (retry) on win_rel for steps
"interactive_ui_tests, browser_tests".
It's a second try, previously, steps "interactive_ui_tests, browser_tests"
failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-07-31 15:58:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/10832011/2001

[commit-bot: I haz the power, 2012-07-31 19:20:46]

Try job failure for 10832011-2001 (retry) on win_rel for steps
"interactive_ui_tests, browser_tests".
It's a second try, previously, steps "interactive_ui_tests, browser_tests"
failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[sky, 2012-08-01 17:18:12]

I ran into a crash with the first version of this patch. The crash would happen
if when detaching a child view of the HWND had focus. This would mean when the
view was destroyed the focusmanager wouldn't be told (because there was no path
to the focus manager once detached) and the FocusManager would be left holding a
potentially deleted view. The fix is to clear out focus when detaching if a
child of the HWND has focus.

Ben, take another look?

[Ben Goodger (Google), 2012-08-01 17:21:57]

lgtm

[commit-bot: I haz the power, 2012-08-01 17:44:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/10832011/11002

[commit-bot: I haz the power, 2012-08-01 19:24:03]

Change committed as 149459