Set view property of pointer events

Set view property of pointer events

Set view property of touch pointer events to its
target's window and also set this view property
while cloning them for boundary events.
For mouse events it is already set from the
corresponding mouse event.

BUG=611053
Committed: https://crrev.com/79c4be60e6f02ca8fd9b982e1d057585e3784475
Cr-Commit-Position: refs/heads/master@{#393945}

[Navid Zolghadr, 2016-05-13 16:30:45]

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

[dtapuska, 2016-05-13 16:57:54]

lgtm % type of one argument

https://codereview.chromium.org/1976603006/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/events/PointerEventFactory.h (right):

https://codereview.chromium.org/1976603006/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/events/PointerEventFactory.h:41:
LocalDOMWindow*);
This should be a DOMWindow* and not a LocalDOMWindow.

Have a look at the UIEvent constructor. It will take an AbstractView ==
DOMWindow.

[Navid Zolghadr, 2016-05-16 13:34:41]

https://codereview.chromium.org/1976603006/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/events/PointerEventFactory.h (right):

https://codereview.chromium.org/1976603006/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/events/PointerEventFactory.h:41:
LocalDOMWindow*);
On 2016/05/13 16:57:53, dtapuska wrote:
> This should be a DOMWindow* and not a LocalDOMWindow.
> 
> Have a look at the UIEvent constructor. It will take an AbstractView ==
> DOMWindow.

Done.

[dtapuska, 2016-05-16 13:37:20]

On 2016/05/16 13:34:41, Navid Zolghadr wrote:
>
https://codereview.chromium.org/1976603006/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/events/PointerEventFactory.h (right):
> 
>
https://codereview.chromium.org/1976603006/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/events/PointerEventFactory.h:41:
> LocalDOMWindow*);
> On 2016/05/13 16:57:53, dtapuska wrote:
> > This should be a DOMWindow* and not a LocalDOMWindow.
> > 
> > Have a look at the UIEvent constructor. It will take an AbstractView ==
> > DOMWindow.
> 
> Done.

lgtm

[mustaq, 2016-05-16 14:49:16]

lgtm

[Navid Zolghadr, 2016-05-16 14:49:43]

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

[Rick Byers, 2016-05-16 16:00:33]

LGTM

[Navid Zolghadr, 2016-05-16 17:06:41]

The CQ bit was checked by nzolghadr@chromium.org

[commit-bot: I haz the power, 2016-05-16 17:07:09]

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

[commit-bot: I haz the power, 2016-05-16 17:17:37]

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

[commit-bot: I haz the power, 2016-05-16 17:17:38]

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

[Navid Zolghadr, 2016-05-16 20:44:21]

The CQ bit was checked by nzolghadr@chromium.org

[Navid Zolghadr, 2016-05-16 20:44:22]

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

[commit-bot: I haz the power, 2016-05-16 20:44:59]

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

[commit-bot: I haz the power, 2016-05-16 21:53:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-16 21:55:32]

Description was changed from

==========
Set view property of pointer events

Set view property of touch pointer events to its
target's window and also set this view property
while cloning them for boundary events.
For mouse events it is already set from the
corresponding mouse event.

BUG=611053
==========

to

==========
Set view property of pointer events

Set view property of touch pointer events to its
target's window and also set this view property
while cloning them for boundary events.
For mouse events it is already set from the
corresponding mouse event.

BUG=611053

Committed: https://crrev.com/79c4be60e6f02ca8fd9b982e1d057585e3784475
Cr-Commit-Position: refs/heads/master@{#393945}
==========

[commit-bot: I haz the power, 2016-05-16 21:55:33]

Patchset 3 (id:??) landed as
https://crrev.com/79c4be60e6f02ca8fd9b982e1d057585e3784475
Cr-Commit-Position: refs/heads/master@{#393945}