Refactor FindScrollLayerForDeviceViewportPoint to use scroll nodes over layer ids

Refactor FindScrollLayerForDeviceViewportPoint to use scroll nodes over layer ids

This patch refactors FindScrollLayerForDeviceViewportPoint as
FindScrollNodeForDeviceViewportPoint which both simplifies code and lets
us remove two users of ScrollNode's owning_layer_id.

BUG=693740
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2824683002
Cr-Commit-Position: refs/heads/master@{#465677}
Committed: https://chromium.googlesource.com/chromium/src/+/dfd089f1bd1aa1d8512741f2a103d0b87684f2e4

[pdr., 2017-04-15 23:25:02]

pdr@chromium.org changed reviewers:
+ weiliangc@chromium.org, wkorman@chromium.org

[wkorman, 2017-04-17 17:32:52]

lgtm

[weiliangc, 2017-04-18 15:24:43]

lgtm

[pdr., 2017-04-18 17:30:23]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2017-04-18 17:31:00]

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

[commit-bot: I haz the power, 2017-04-18 17:31:01]

This CL has an open dependency (Issue 2816063003 Patch 60001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[pdr., 2017-04-19 17:35:34]

The CQ bit was checked by pdr@chromium.org

[pdr., 2017-04-19 17:35:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
weiliangc@chromium.org, wkorman@chromium.org
Link to the patchset: https://codereview.chromium.org/2824683002/#ps20001
(title: "Rebase from space")

[commit-bot: I haz the power, 2017-04-19 17:35:59]

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-04-19 18:29:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492623334355580,
"parent_rev": "28695cd0fcce0beddcaa7b02d224d63c7bf9f7a5", "commit_rev":
"dfd089f1bd1aa1d8512741f2a103d0b87684f2e4"}

[commit-bot: I haz the power, 2017-04-19 18:29:44]

Description was changed from

==========
Refactor FindScrollLayerForDeviceViewportPoint to use scroll nodes over layer
ids

This patch refactors FindScrollLayerForDeviceViewportPoint as
FindScrollNodeForDeviceViewportPoint which both simplifies code and lets
us remove two users of ScrollNode's owning_layer_id.

BUG=693740
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Refactor FindScrollLayerForDeviceViewportPoint to use scroll nodes over layer
ids

This patch refactors FindScrollLayerForDeviceViewportPoint as
FindScrollNodeForDeviceViewportPoint which both simplifies code and lets
us remove two users of ScrollNode's owning_layer_id.

BUG=693740
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2824683002
Cr-Commit-Position: refs/heads/master@{#465677}
Committed:
https://chromium.googlesource.com/chromium/src/+/dfd089f1bd1aa1d8512741f2a103...
==========

[commit-bot: I haz the power, 2017-04-19 18:29:45]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/dfd089f1bd1aa1d8512741f2a103...