Clean up GestureEventDetails constructors and fix unit tests.

Clean up GestureEventDetails constructors and fix unit tests.

Reland the code change for Issue 573963005: Clean up GestureEventDetails
constructors and fix unit tests:
https://codereview.chromium.org/573963005/.

The code does not change, just merge two commits into one.

BUG=350942
TBR=tdresser@chromium.org,jdduke@chromium.org,sadrul@chromium.org,sky@chromium.org

Committed: https://crrev.com/46f76555938808dd916c4e2d5c0d7de1333a821d
Cr-Commit-Position: refs/heads/master@{#295706}

[lanwei, 2014-09-19 04:11:08]

The CQ bit was checked by lanwei@chromium.org

[commit-bot: I haz the power, 2014-09-19 04:12:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/581963004/20001

[commit-bot: I haz the power, 2014-09-19 04:12:01]

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

[commit-bot: I haz the power, 2014-09-19 04:12:03]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lanwei, 2014-09-19 04:33:45]

lanwei@chromium.org changed reviewers:
+ sky@chromium.org

[lanwei, 2014-09-19 04:33:55]

The CQ bit was checked by lanwei@chromium.org

[commit-bot: I haz the power, 2014-09-19 04:35:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/581963004/20001

[commit-bot: I haz the power, 2014-09-19 04:35:09]

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

[commit-bot: I haz the power, 2014-09-19 04:35:11]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lanwei, 2014-09-19 04:36:52]

Patchset #1 (id:1) has been deleted

[lanwei, 2014-09-19 04:39:28]

sky@, can you please review this issue again? They revert it by mistake, because
they thought it broke wm_unittests on XP Tests (3), but it did not.
Thank you.

[tdresser, 2014-09-19 14:22:14]

On 2014/09/19 04:39:28, lanwei wrote:
> sky@, can you please review this issue again? They revert it by mistake,
because
> they thought it broke wm_unittests on XP Tests (3), but it did not.
> Thank you.

LGTM (With the TBR, I think and LGTM from any committer will work?)

[lanwei, 2014-09-19 14:23:47]

The CQ bit was checked by lanwei@chromium.org

[commit-bot: I haz the power, 2014-09-19 14:24:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/581963004/20001

[commit-bot: I haz the power, 2014-09-19 14:52:56]

Committed patchset #1 (id:20001) as c8275a64567540d2ebb0797b5dee581df8777993

[commit-bot: I haz the power, 2014-09-19 14:53:29]

Patchset 1 (id:??) landed as
https://crrev.com/46f76555938808dd916c4e2d5c0d7de1333a821d
Cr-Commit-Position: refs/heads/master@{#295706}