Removed pinch viewport scroll offset distribution

Removed pinch viewport scroll offset distribution

The associated change in Blink makes the pinch viewport a proper
ScrollableArea meaning the normal path for synchronizing layer scroll
offsets is used.

This is a 2 sided patch, the other CL:
https://codereview.chromium.org/199253002/

BUG=349941
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260105

[aelias_OOO_until_Jul13, 2014-03-25 00:09:23]

https://codereview.chromium.org/210543002/diff/1/cc/layers/layer.cc
File cc/layers/layer.cc (right):

https://codereview.chromium.org/210543002/diff/1/cc/layers/layer.cc#newcode682
cc/layers/layer.cc:682: scroll_offset =
layer_tree_host()->DistributeScrollOffsetToViewports(
It looks like this method just does "return offset" when called on the old pinch
flow, so can we just delete the whole thing now instead?

[bokan, 2014-03-25 00:22:20]

On 2014/03/25 00:09:23, aelias wrote:
> https://codereview.chromium.org/210543002/diff/1/cc/layers/layer.cc
> File cc/layers/layer.cc (right):
> 
> https://codereview.chromium.org/210543002/diff/1/cc/layers/layer.cc#newcode682
> cc/layers/layer.cc:682: scroll_offset =
> layer_tree_host()->DistributeScrollOffsetToViewports(
> It looks like this method just does "return offset" when called on the old
pinch
> flow, so can we just delete the whole thing now instead?

Right, forgot that it's not used in old-style pinch. Done.

[aelias_OOO_until_Jul13, 2014-03-25 00:27:33]

lgtm

[bokan, 2014-03-27 12:36:33]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 12:36:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/20001

[commit-bot: I haz the power, 2014-03-27 14:03:36]

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

[commit-bot: I haz the power, 2014-03-27 14:03:37]

Try jobs failed on following builders:
  tryserver.chromium on mac_chromium_rel

[bokan, 2014-03-27 14:22:48]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 14:23:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-27 14:47:33]

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

[commit-bot: I haz the power, 2014-03-27 14:47:34]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_clang_dbg
  tryserver.chromium on linux_chromium_rel

[bokan, 2014-03-27 14:57:31]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 14:57:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-27 15:17:02]

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

[commit-bot: I haz the power, 2014-03-27 15:17:03]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_clang_dbg

[bokan, 2014-03-27 15:42:11]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 15:42:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-27 15:53:57]

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

[commit-bot: I haz the power, 2014-03-27 15:53:58]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_clang_dbg

[bokan, 2014-03-27 16:33:39]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 16:35:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-27 16:42:37]

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

[commit-bot: I haz the power, 2014-03-27 16:42:37]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

[bokan, 2014-03-27 18:18:13]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 18:19:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-27 20:53:03]

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

[commit-bot: I haz the power, 2014-03-27 20:53:04]

Retried try job too often on win_rel for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[bokan, 2014-03-27 21:01:56]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-27 21:05:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-27 22:29:09]

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

[commit-bot: I haz the power, 2014-03-27 22:29:09]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[bokan, 2014-03-28 11:20:39]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2014-03-28 11:22:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bokan@chromium.org/210543002/40001

[commit-bot: I haz the power, 2014-03-28 11:24:21]

Change committed as 260105