Remove the pointer event capturing flag

Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776
TBR=avi@chromium.org

Committed: https://crrev.com/0d9eef755924350100e4ce8d06ea06fac9511a3e
Cr-Commit-Position: refs/heads/master@{#434670}

[Navid Zolghadr, 2016-11-23 16:51:04]

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

[commit-bot: I haz the power, 2016-11-23 16:51:30]

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

[Navid Zolghadr, 2016-11-23 16:53:20]

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

[mustaq, 2016-11-23 17:14:08]

lgtm

https://codereview.chromium.org/2525973002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/input/EventHandler.cpp (right):

https://codereview.chromium.org/2525973002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/input/EventHandler.cpp:1180:
m_pointerEventManager->getMouseCapturingNode()) {
Nit: fix indentation.

[Navid Zolghadr, 2016-11-23 17:24:34]

https://codereview.chromium.org/2525973002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/input/EventHandler.cpp (right):

https://codereview.chromium.org/2525973002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/input/EventHandler.cpp:1180:
m_pointerEventManager->getMouseCapturingNode()) {
On 2016/11/23 17:14:08, mustaq wrote:
> Nit: fix indentation.

This is correct I believe. I used "git cl format" and that moved it a little. I
believe it is because it is residual of last = as oppose to simple extra
condition.

[mustaq, 2016-11-23 17:26:10]

On 2016/11/23 17:24:34, Navid Zolghadr wrote:
>
https://codereview.chromium.org/2525973002/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/input/EventHandler.cpp (right):
> 
>
https://codereview.chromium.org/2525973002/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/input/EventHandler.cpp:1180:
> m_pointerEventManager->getMouseCapturingNode()) {
> On 2016/11/23 17:14:08, mustaq wrote:
> > Nit: fix indentation.
> 
> This is correct I believe. I used "git cl format" and that moved it a little.
I
> believe it is because it is residual of last = as oppose to simple extra
> condition.

Thanks for confirming, still LGTM.

[commit-bot: I haz the power, 2016-11-23 18:30:54]

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

[commit-bot: I haz the power, 2016-11-23 18:30:55]

Dry run: 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, 2016-11-23 19:14:52]

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

[Navid Zolghadr, 2016-11-24 15:26:34]

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

[Navid Zolghadr, 2016-11-24 15:26:34]

avi@chromium.org: Please review changes in 

content/*

[Rick Byers, 2016-11-25 01:43:37]

LGTM

Once pointer events hits stable we might hit a few web compat issues where this
would be a useful tool, but we can just use it in Chrome 55 or 56.  It seems
unlikely that we'll still have any use for it by the time 57 is stable (and
worst case we can easily add it back for awhile longer).

[Navid Zolghadr, 2016-11-28 15:33:28]

Description was changed from

==========
Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776
==========

to

==========
Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776

TBR=avi@chromium.org
==========

[Navid Zolghadr, 2016-11-28 15:33:35]

The CQ bit was checked by nzolghadr@chromium.org

[commit-bot: I haz the power, 2016-11-28 15:33:47]

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, 2016-11-28 17:04:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480347215944690, "parent_rev":
"d8e7080016ec2c019df5b1440ffed5dcd24aa06c", "commit_rev":
"a0478a8326d5502b34cbf3be5c902f9055f12886"}

[commit-bot: I haz the power, 2016-11-28 17:05:08]

Description was changed from

==========
Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776

TBR=avi@chromium.org
==========

to

==========
Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776

TBR=avi@chromium.org
==========

[commit-bot: I haz the power, 2016-11-28 17:05:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-28 17:07:16]

Description was changed from

==========
Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776

TBR=avi@chromium.org
==========

to

==========
Remove the pointer event capturing flag

This was a flag for testing purposes and
was added as part of crbug.com/640700. This
CL removes that flag which is no longer needed.

BUG=642776

TBR=avi@chromium.org

Committed: https://crrev.com/0d9eef755924350100e4ce8d06ea06fac9511a3e
Cr-Commit-Position: refs/heads/master@{#434670}
==========

[commit-bot: I haz the power, 2016-11-28 17:07:18]

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

[Avi (use Gerrit), 2016-11-28 18:04:33]

LGTM