Remove FrameView::m_inProgrammaticScroll and related plumbing.

Remove FrameView::m_inProgrammaticScroll and related plumbing.

It seems this field only exists so that FrameView::setWasScrolledByUser can say
"oh, you didn't really mean to do that".

With this patch we no longer call setWasScrolledByUser(true) from
FrameView::scrollPositionChanged, which handles both user and programmatic
scrolls, and instead call it only for actual user scrolls. EventHandler was
already doing this in many cases.

This happens to revert the effects of http://crrev.com/16254006, which I think
are no longer needed (aelias thinks it was for http://crbug.com/165317).

Not having to update m_inProgrammaticScroll simplifies other scrolling changes,
such as RootFrameViewport and root layer scrolling.

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

[skobes, 2015-04-30 15:24:37]

Patchset #1 (id:1) has been deleted

[skobes, 2015-04-30 16:02:53]

skobes@chromium.org changed reviewers:
+ dsinclair@chromium.org

[dsinclair, 2015-05-01 02:06:12]

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

[dsinclair, 2015-05-01 02:06:13]

lgtm

+pdr for Source/web

[pdr., 2015-05-01 02:07:20]

On 2015/05/01 at 02:06:13, dsinclair wrote:
> lgtm
> 
> +pdr for Source/web

LGTM

[skobes, 2015-05-01 04:40:04]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2015-05-01 04:40:31]

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

[commit-bot: I haz the power, 2015-05-01 04:58:33]

Committed patchset #1 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194786