Fix a long standing bug with the browser window losing focus when the user scrolled...
commit9d7d7494a5d2ceedde75f88ecd0bb6866044eea0
authorananta <ananta@chromium.org>
Sat, 4 Oct 2014 01:18:38 +0000 (3 18:18 -0700)
committerCommit bot <commit-bot@chromium.org>
Sat, 4 Oct 2014 01:18:52 +0000 (4 01:18 +0000)
tree9504164aa2555319360b69f88bfc49eb935c8e5b
parentfc98010259f01b13fc612e85bc0f4c2b2cde6e60
Fix a long standing bug with the browser window losing focus when the user scrolled on the page using certain trackpad drivers.

The bug occurs because the browser window receives a WM_MOUSEACTIVATE message when the trackpad is
operated on the web page. There is code in the HWNDMessageHandler::OnMouseActivate function which
checks if the window under the cursor is a child window and if yes it causes the focus manager
to lose activation. This code was added to support activation in windowed NPAPI plugins.

It causes this issue because the child window in question here is the legacy RenderWidgetHostHWND
class.

Fix for now is to ignore transparent child windows. Added a TODO in HWNDMessageHandler to remove the
child window related code once we deprecate NPAPI plugins.

BUG=351995

Review URL: https://codereview.chromium.org/626183002

Cr-Commit-Position: refs/heads/master@{#298151}
ui/views/win/hwnd_message_handler.cc