OnPointerMessage no longer nukes the active focus (Win8/Desktop).

Disable force focus call from pointer events in Chrome/Win/Desktop

This code was added in an attempt to make the on-screen-keyboard function in desktop chrome. By coincidence, it messes up popup menus/drop-down boxes. Removing the code.

Issue 105637 tracks the OSK issue. We'll need to revisit/re-enable this code when we work on that.

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

[sky, 2012-09-20 19:29:17]

Ananta has been tweaking this code, he should look at this too. If we're going
to do this, just remove the code. No point in the #if 0.

[girard, 2012-09-21 02:29:33]

On 2012/09/20 19:29:17, sky wrote:
> Ananta has been tweaking this code, he should look at this too. If we're going
> to do this, just remove the code. No point in the #if 0.

I was wondering about that (commenting out the code, using #if 0, or removing
it) and thought that I should follow the example set in
RenderWidgetHostViewWin::CreateWnd (where the OSK creation logic was #if 0'd
out).

Happy to get rid of it completely though.  I'll get that CL up tomorrow.

[girard, 2012-09-21 13:41:40]

As per sky's comment, I've removed the dead code.

[sky, 2012-09-21 16:19:32]

Again, Anata needs to comment on this as he has other patches in this area.

[ananta, 2012-09-21 17:01:08]

On 2012/09/21 16:19:32, sky wrote:
> Again, Anata needs to comment on this as he has other patches in this area.

LGTM

[commit-bot: I haz the power, 2012-09-21 17:30:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/girard@chromium.org/10825233/4003

[commit-bot: I haz the power, 2012-09-21 17:30:25]

Failed to apply patch for
content/browser/renderer_host/render_widget_host_view_win.cc:
While running patch -p1 --forward --force;
patching file content/browser/renderer_host/render_widget_host_view_win.cc
Hunk #1 FAILED at 2691.
1 out of 1 hunk FAILED -- saving rejects to file
content/browser/renderer_host/render_widget_host_view_win.cc.rej

[sky, 2012-09-21 18:34:00]

LGTM

[girard, 2012-09-21 18:38:09]

Minor update.  After the submit hit a merge conflict, I rebased and found out
that the entire OnPointerEvent function was now empty.  Removed the function.

[commit-bot: I haz the power, 2012-09-21 18:53:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/girard@chromium.org/10825233/4007

[commit-bot: I haz the power, 2012-09-21 20:53:42]

Sorry for I got bad news for ya.
Compile failed with a clobber build.
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[girard, 2012-09-22 02:28:26]

On 2012/09/21 20:53:42, I haz the power (commit-bot) wrote:
> Sorry for I got bad news for ya.
> Compile failed with a clobber build.
> Your code is likely broken or HEAD is junk. Please ensure your
> code is not broken then alert the build sheriffs.
> Look at the try server FAQ for more details.

"There is not enough space on the disk." - I think I'll try again.

[commit-bot: I haz the power, 2012-09-22 02:28:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/girard@chromium.org/10825233/4007

[commit-bot: I haz the power, 2012-09-22 06:16:34]

Change committed as 158173