Ignore pointer capture target while calculating click target

Ignore pointer capture target while calculating click target

After https://codereview.chromium.org/2681443003
it turns out we are creating some regressions.
So this change keeps the essence of the original
change for the purpose of keeping the metric but
ignores the target that is calculated from the
captured target and use the original click target
regardless of the capturing.

BUG=701599, 699933
Review-Url: https://codereview.chromium.org/2755753003
Cr-Commit-Position: refs/heads/master@{#458091}
Committed: https://chromium.googlesource.com/chromium/src/+/ddb951c0cc2912acd57e4238677b82e6360ac1ad

[Navid Zolghadr, 2017-03-16 16:43:34]

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

[Navid Zolghadr, 2017-03-16 16:43:49]

nzolghadr@chromium.org changed reviewers:
+ dtapuska@chromium.org, mustaq@chromium.org

[commit-bot: I haz the power, 2017-03-16 16:44:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-16 19:04:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-16 19:04:56]

Dry run: This issue passed the CQ dry run.

[dtapuska, 2017-03-17 18:40:51]

lgtm

[Navid Zolghadr, 2017-03-17 18:42:45]

nzolghadr@chromium.org changed reviewers:
+ bokan@chromium.org

[bokan, 2017-03-17 19:03:19]

rs lgtm based on Dave's review

[Navid Zolghadr, 2017-03-17 19:12:23]

The CQ bit was checked by nzolghadr@chromium.org

[commit-bot: I haz the power, 2017-03-17 19:12:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-17 20:33:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-17 20:33:59]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Navid Zolghadr, 2017-03-17 20:37:43]

The CQ bit was checked by nzolghadr@chromium.org

[commit-bot: I haz the power, 2017-03-17 20:38:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-17 20:45:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-17 20:45:49]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Navid Zolghadr, 2017-03-20 14:18:55]

The CQ bit was checked by nzolghadr@chromium.org

[commit-bot: I haz the power, 2017-03-20 14:19:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-20 16:45:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490019535224450, "parent_rev":
"3a63ad337f6b5dd926292b496f4cb9bb14d4594d", "commit_rev":
"ddb951c0cc2912acd57e4238677b82e6360ac1ad"}

[commit-bot: I haz the power, 2017-03-20 16:46:33]

Description was changed from

==========
Ignore pointer capture target while calculating click target

After https://codereview.chromium.org/2681443003
it turns out we are creating some regressions.
So this change keeps the essence of the original
change for the purpose of keeping the metric but
ignores the target that is calculated from the
captured target and use the original click target
regardless of the capturing.

BUG=701599, 699933
==========

to

==========
Ignore pointer capture target while calculating click target

After https://codereview.chromium.org/2681443003
it turns out we are creating some regressions.
So this change keeps the essence of the original
change for the purpose of keeping the metric but
ignores the target that is calculated from the
captured target and use the original click target
regardless of the capturing.

BUG=701599, 699933

Review-Url: https://codereview.chromium.org/2755753003
Cr-Commit-Position: refs/heads/master@{#458091}
Committed:
https://chromium.googlesource.com/chromium/src/+/ddb951c0cc2912acd57e4238677b...
==========

[commit-bot: I haz the power, 2017-03-20 16:46:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ddb951c0cc2912acd57e4238677b...

[johnme, 2017-03-21 12:44:09]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2764813004/ by johnme@chromium.org.

The reason for reverting is: Tentative revert. Hit testing broke in the
following tests, and this patch looks the most closely related:

AccessibilityHitTestingBrowserTest.HitTestingInIframes
AccessibilityHitTestingBrowserTest.CachingAsyncHitTestingInIframes.

[Navid Zolghadr, 2017-03-21 17:15:49]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2762913003/ by nzolghadr@chromium.org.

The reason for reverting is: Reverting all the changes related to click target
in capturing as they are splitted across multiple commits..