Fix fake mouse move event on scroll

Fix fake mouse move event on scroll

We made LayoutObject::previousPaintInvalidationRect() private but
DeprecatedPaintLayerScrollArea is a friend of LayoutObject, so missed
the change. This caused incorrect fake mouse move event on composited
scroll.

Will fix the friend access issue in
https://codereview.chromium.org/1306993002/.

BUG=516016
TEST=fast/events/overflow-composited-scroll-fake-mouse-move.html

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201028

[Xianzhu, 2015-08-21 21:23:21]

wangxianzhu@chromium.org changed reviewers:
+ pdr@chromium.org

[pdr., 2015-08-21 21:34:09]

LGTM

https://codereview.chromium.org/1309983002/diff/1/LayoutTests/fast/events/ove...
File LayoutTests/fast/events/overflow-composited-scroll-fake-mouse-move.html
(right):

https://codereview.chromium.org/1309983002/diff/1/LayoutTests/fast/events/ove...
LayoutTests/fast/events/overflow-composited-scroll-fake-mouse-move.html:29:
setTimeout(fail, 1000);
Lets just let the test timeout instead of making it timeout explicitly.

[Xianzhu, 2015-08-21 21:40:52]

https://codereview.chromium.org/1309983002/diff/1/LayoutTests/fast/events/ove...
File LayoutTests/fast/events/overflow-composited-scroll-fake-mouse-move.html
(right):

https://codereview.chromium.org/1309983002/diff/1/LayoutTests/fast/events/ove...
LayoutTests/fast/events/overflow-composited-scroll-fake-mouse-move.html:29:
setTimeout(fail, 1000);
On 2015/08/21 21:34:09, pdr wrote:
> Lets just let the test timeout instead of making it timeout explicitly.

Done.

[Xianzhu, 2015-08-21 21:40:55]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2015-08-21 21:40:55]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/1309983002/#ps20001
(title: "Remove explicit timeout on failure")

[commit-bot: I haz the power, 2015-08-21 21:41:12]

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

[commit-bot: I haz the power, 2015-08-21 21:41:13]

This CL has an open dependency (Issue 1302993003 Patch 100001). Please resolve
the dependency and try again.

[pdr., 2015-08-21 21:47:22]

On 2015/08/21 at 21:41:13, commit-bot wrote:
> This CL has an open dependency (Issue 1302993003 Patch 100001). Please resolve
the dependency and try again.

Hmm, what is this

[Xianzhu, 2015-08-21 21:55:41]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2015-08-21 21:55:42]

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

[commit-bot: I haz the power, 2015-08-21 21:56:07]

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

[Xianzhu, 2015-08-21 21:57:16]

On 2015/08/21 21:47:22, pdr wrote:
> On 2015/08/21 at 21:41:13, commit-bot wrote:
> > This CL has an open dependency (Issue 1302993003 Patch 100001). Please
resolve
> the dependency and try again.
> 
> Hmm, what is this

C depends on B (this one) and A (unrelated to B) and I made the dependency chain
C -> B -> A.

[commit-bot: I haz the power, 2015-08-22 00:16:30]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201028