Ignore gesture sequences that have no target.

Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggered by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
Committed: https://crrev.com/2816a742570a0f68237ec9fd6ab31320c36a75e1
Cr-Commit-Position: refs/heads/master@{#381540}

[wjmaclean, 2016-03-16 18:56:48]

The CQ bit was checked by wjmaclean@chromium.org to run a CQ dry run

[wjmaclean, 2016-03-16 18:57:07]

Description was changed from

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggerd by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

to

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggerd by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

[wjmaclean, 2016-03-16 18:57:07]

wjmaclean@chromium.org changed reviewers:
+ tdresser@chromium.org

[commit-bot: I haz the power, 2016-03-16 18:57:33]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1810693002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1810693002/1

[wjmaclean, 2016-03-16 18:57:35]

I'm running this on the bots to sanity check it, but please take a look when you
get a moment.

[wjmaclean, 2016-03-16 18:59:57]

Description was changed from

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggerd by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

to

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggered by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

[tdresser, 2016-03-16 19:23:00]

LGTM % nit.

https://codereview.chromium.org/1810693002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/render_widget_host_input_event_router.cc
(right):

https://codereview.chromium.org/1810693002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/render_widget_host_input_event_router.cc:175: //
Ignore this gesture sequence as no target is available.
Add a TODO with a reference to the bug to get rid of this.

[wjmaclean, 2016-03-16 19:55:59]

Patchset #2 (id:20001) has been deleted

[wjmaclean, 2016-03-16 19:56:41]

The CQ bit was checked by wjmaclean@chromium.org

[wjmaclean, 2016-03-16 19:56:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdresser@chromium.org
Link to the patchset: https://codereview.chromium.org/1810693002/#ps40001
(title: "Add TODO with bug.")

[commit-bot: I haz the power, 2016-03-16 19:57:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1810693002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1810693002/40001

[commit-bot: I haz the power, 2016-03-16 21:00:20]

Description was changed from

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggered by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

to

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggered by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

[commit-bot: I haz the power, 2016-03-16 21:00:21]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-16 21:03:08]

Description was changed from

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggered by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319
==========

to

==========
Ignore gesture sequences that have no target.

This CL implements a speculative fix for the associated bug, on the
theory that sequences associated with context menus (triggered by
GestureLongPress) may not be properly suppressed on Windows, leading
to the observation of sequences without any target in
RenderWidgetHostInputEventRouter. If this hypothesis is correct, then
we should be correct in ignoring such sequences, and this CL implements
that.

This CL also removes the code for collecting debug data introduced in
https://codereview.chromium.org/1783533002/.

BUG=592319

Committed: https://crrev.com/2816a742570a0f68237ec9fd6ab31320c36a75e1
Cr-Commit-Position: refs/heads/master@{#381540}
==========

[commit-bot: I haz the power, 2016-03-16 21:03:09]

Patchset 2 (id:??) landed as
https://crrev.com/2816a742570a0f68237ec9fd6ab31320c36a75e1
Cr-Commit-Position: refs/heads/master@{#381540}