Include the source_to_parent offset when removing offset applied by main.

Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=LayerTreeHostCommonTest.StickyPositionTopScrollParent, compositing/overflow/composited-sticky-element.html
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/d2ae03fccb90a1de08a12be2c9d81cf4ff04f713
Cr-Commit-Position: refs/heads/master@{#431898}

[flackr, 2016-11-10 14:16:41]

Description was changed from

==========
Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=compositing/overflow/composited-sticky-element.html
==========

to

==========
Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=compositing/overflow/composited-sticky-element.html
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[flackr, 2016-11-10 14:18:06]

flackr@chromium.org changed reviewers:
+ ajuma@chromium.org

[flackr, 2016-11-10 14:18:06]

Hey,

This is the fix for the scroll parent case. Can you take a look?

Thanks,
Rob

[ajuma, 2016-11-10 15:17:09]

On 2016/11/10 14:18:06, flackr wrote:
> Hey,
> 
> This is the fix for the scroll parent case. Can you take a look?
> 
> Thanks,
> Rob

Fix looks good, but please add a cc unit test as well.

[flackr, 2016-11-11 15:37:10]

Description was changed from

==========
Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=compositing/overflow/composited-sticky-element.html
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=LayerTreeHostCommonTest.StickyPositionTopScrollParent,
compositing/overflow/composited-sticky-element.html
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[flackr, 2016-11-11 15:37:20]

On 2016/11/10 at 15:17:09, ajuma wrote:
> On 2016/11/10 14:18:06, flackr wrote:
> > Hey,
> > 
> > This is the fix for the scroll parent case. Can you take a look?
> > 
> > Thanks,
> > Rob
> 
> Fix looks good, but please add a cc unit test as well.

Done.

[weiliangc, 2016-11-14 15:05:33]

weiliangc@chromium.org changed reviewers:
+ weiliangc@chromium.org

[weiliangc, 2016-11-14 15:05:34]

LGTM

[flackr, 2016-11-14 15:10:13]

The CQ bit was checked by flackr@chromium.org

[commit-bot: I haz the power, 2016-11-14 15:10:32]

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-14 17:12:00]

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

[commit-bot: I haz the power, 2016-11-14 17:12:01]

Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_TIMED_OUT, no build URL)

[flackr, 2016-11-14 18:14:50]

The CQ bit was checked by flackr@chromium.org

[commit-bot: I haz the power, 2016-11-14 18:15:55]

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-14 19:22:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-14 20:09:56]

Description was changed from

==========
Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=LayerTreeHostCommonTest.StickyPositionTopScrollParent,
compositing/overflow/composited-sticky-element.html
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Include the source_to_parent offset when removing offset applied by main.

BUG=663642
TEST=LayerTreeHostCommonTest.StickyPositionTopScrollParent,
compositing/overflow/composited-sticky-element.html
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/d2ae03fccb90a1de08a12be2c9d81cf4ff04f713
Cr-Commit-Position: refs/heads/master@{#431898}
==========

[commit-bot: I haz the power, 2016-11-14 20:09:57]

Patchset 2 (id:??) landed as
https://crrev.com/d2ae03fccb90a1de08a12be2c9d81cf4ff04f713
Cr-Commit-Position: refs/heads/master@{#431898}