Add blink::WebInputEvent::IsPinchGestureEventType().

Add blink::WebInputEvent::IsPinchGestureEventType().

Review-Url: https://codereview.chromium.org/2861373004
Cr-Commit-Position: refs/heads/master@{#472185}
Committed: https://chromium.googlesource.com/chromium/src/+/3df10ca837b3d45b1b8a3f4da796d40f595158e8

[Lei Zhang, 2017-05-08 06:26:03]

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

[commit-bot: I haz the power, 2017-05-08 06:26:26]

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-05-08 10:25:32]

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

[commit-bot: I haz the power, 2017-05-08 10:25:32]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-05-08 19:10:19]

thestig@chromium.org changed reviewers:
+ dtapuska@chromium.org

[Lei Zhang, 2017-05-08 19:10:19]

dtapuska: I remember you were changing WebInputEvent last year. Are you the
right person to review the WebInputEvent.h changes?

[dtapuska, 2017-05-08 19:19:42]

On 2017/05/08 19:10:19, Lei Zhang wrote:
> dtapuska: I remember you were changing WebInputEvent last year. Are you the
> right person to review the WebInputEvent.h changes?

lgtm % nit.

[dtapuska, 2017-05-08 19:23:19]

https://codereview.chromium.org/2861373004/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebInputEvent.h (right):

https://codereview.chromium.org/2861373004/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebInputEvent.h:316: static bool
IsPinchGestureEventType(int type) {
can you use WebInputEvent::Type here instead of int?

[Lei Zhang, 2017-05-08 22:49:32]

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

[Lei Zhang, 2017-05-08 22:50:45]

https://codereview.chromium.org/2861373004/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebInputEvent.h (right):

https://codereview.chromium.org/2861373004/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebInputEvent.h:316: static bool
IsPinchGestureEventType(int type) {
On 2017/05/08 19:23:19, dtapuska wrote:
> can you use WebInputEvent::Type here instead of int?

Done. I fixed the other IsFooType() methods above too.

[Lei Zhang, 2017-05-08 22:51:51]

thestig@chromium.org changed reviewers:
+ lazyboy@chromium.org, thakis@chromium.org

[Lei Zhang, 2017-05-08 22:51:51]

OWNERS:

lazyboy: components, extensions
thakis: third_party/WebKit, ui

[commit-bot: I haz the power, 2017-05-08 22:57:37]

Dry run: CQ is trying da patch.

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

[lazyboy, 2017-05-08 22:59:27]

extensions + guest view lgtm

[commit-bot: I haz the power, 2017-05-09 00:43:12]

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

[commit-bot: I haz the power, 2017-05-09 00:43:12]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Lei Zhang, 2017-05-15 22:42:20]

On 2017/05/08 22:51:51, Lei Zhang wrote:
> thakis: third_party/WebKit, ui

ping.

[Nico, 2017-05-15 22:46:42]

lgtm

Calling blink code from the browser is a bit questionable due to the chrome.dll
/ chrome_child.dll split on Windows, but I suppose there's precedent, and it's
inline so it won't cause blink to get linked into the parent dll.

[Lei Zhang, 2017-05-16 01:00:41]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2017-05-16 01:00:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtapuska@chromium.org
Link to the patchset: https://codereview.chromium.org/2861373004/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2017-05-16 01:01:58]

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-05-16 03:31:29]

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

[commit-bot: I haz the power, 2017-05-16 03:31:31]

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

[Lei Zhang, 2017-05-16 05:58:02]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-05-16 05:58:22]

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-05-16 08:03:18]

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

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

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

[Lei Zhang, 2017-05-16 16:33:25]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-05-16 16:34:14]

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-05-16 19:35:49]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494952405201170,
"parent_rev": "7cf21a287bc149f34de83c2bb57b61e6d206fa6b", "commit_rev":
"3df10ca837b3d45b1b8a3f4da796d40f595158e8"}

[commit-bot: I haz the power, 2017-05-16 19:36:01]

Description was changed from

==========
Add blink::WebInputEvent::IsPinchGestureEventType().
==========

to

==========
Add blink::WebInputEvent::IsPinchGestureEventType().

Review-Url: https://codereview.chromium.org/2861373004
Cr-Commit-Position: refs/heads/master@{#472185}
Committed:
https://chromium.googlesource.com/chromium/src/+/3df10ca837b3d45b1b8a3f4da796...
==========

[commit-bot: I haz the power, 2017-05-16 19:36:02]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/3df10ca837b3d45b1b8a3f4da796...