Remove touch-scrolling-mode flag

Remove touch-scrolling-mode flag

Because now we are using throttled async touch model for scroll only,
we will remove touch-scrolling-mode related code.

BUG=350430
Committed: https://crrev.com/220483d0c533c2fa938fd5b150ab03ff77b195a7
Cr-Commit-Position: refs/heads/master@{#314456}

[lanwei, 2015-01-29 16:18:52]

Patchset #1 (id:1) has been deleted

[lanwei, 2015-01-29 18:50:37]

Patchset #1 (id:20001) has been deleted

[lanwei, 2015-01-29 21:19:44]

lanwei@chromium.org changed reviewers:
+ jdduke@chromium.org, rbyers@chromium.org, tdresser@chromium.org

[lanwei, 2015-01-29 21:59:33]

Patchset #1 (id:40001) has been deleted

[jdduke (slow), 2015-01-30 00:12:56]

Yay! lgtm.

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
File content/browser/renderer_host/input/touch_event_queue.cc (right):

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
content/browser/renderer_host/input/touch_event_queue.cc:543: if
(gesture_event.event.type != blink::WebInputEvent::GestureScrollUpdate)
I guess we might as well consolidate this if with the assignment (invert the
condition), also in the |OnGestureEventAck| below.

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
content/browser/renderer_host/input/touch_event_queue.cc:702: if (ack_result ==
INPUT_EVENT_ACK_STATE_CONSUMED) {
Nit: Looks like we can remove this one-line if brace.

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
File content/browser/renderer_host/input/touch_event_queue_unittest.cc (right):

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
content/browser/renderer_host/input/touch_event_queue_unittest.cc:81:
TouchEventQueue::Config config;
Let's just remove this function entirely.

[Alexei Svitkine (slow), 2015-01-30 15:17:27]

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

[Alexei Svitkine (slow), 2015-01-30 15:17:28]

https://codereview.chromium.org/886563005/diff/80001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/886563005/diff/80001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:50609: -  <int value="-1619757314"
label="touch-scrolling-mode"/>
Entries shouldn't be removed from histograms.xml, since they help decode data
from older versions. You can revert the changes to this file.

[lanwei, 2015-02-03 20:44:31]

lanwei@chromium.org changed reviewers:
+ avi@chromium.org

[lanwei, 2015-02-03 20:44:32]

avi@chromium.org: Please review changes in 

Can you please look at all these files?

Thank you.

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
File content/browser/renderer_host/input/touch_event_queue.cc (right):

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
content/browser/renderer_host/input/touch_event_queue.cc:543: if
(gesture_event.event.type != blink::WebInputEvent::GestureScrollUpdate)
On 2015/01/30 00:12:56, jdduke wrote:
> I guess we might as well consolidate this if with the assignment (invert the
> condition), also in the |OnGestureEventAck| below.

Done.

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
content/browser/renderer_host/input/touch_event_queue.cc:702: if (ack_result ==
INPUT_EVENT_ACK_STATE_CONSUMED) {
On 2015/01/30 00:12:56, jdduke wrote:
> Nit: Looks like we can remove this one-line if brace.

Done.

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
File content/browser/renderer_host/input/touch_event_queue_unittest.cc (right):

https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
content/browser/renderer_host/input/touch_event_queue_unittest.cc:81:
TouchEventQueue::Config config;
On 2015/01/30 00:12:56, jdduke wrote:
> Let's just remove this function entirely.

Done.

https://codereview.chromium.org/886563005/diff/80001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/886563005/diff/80001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:50609: -  <int value="-1619757314"
label="touch-scrolling-mode"/>
On 2015/01/30 15:17:28, asvitkine (slow this week) wrote:
> Entries shouldn't be removed from histograms.xml, since they help decode data
> from older versions. You can revert the changes to this file.

Done.

[tdresser, 2015-02-03 21:01:28]

On 2015/02/03 20:44:32, lanwei wrote:
> mailto:avi@chromium.org: Please review changes in 
> 
> Can you please look at all these files?
> 
> Thank you.
> 
>
https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
> File content/browser/renderer_host/input/touch_event_queue.cc (right):
> 
>
https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
> content/browser/renderer_host/input/touch_event_queue.cc:543: if
> (gesture_event.event.type != blink::WebInputEvent::GestureScrollUpdate)
> On 2015/01/30 00:12:56, jdduke wrote:
> > I guess we might as well consolidate this if with the assignment (invert the
> > condition), also in the |OnGestureEventAck| below.
> 
> Done.
> 
>
https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
> content/browser/renderer_host/input/touch_event_queue.cc:702: if (ack_result
==
> INPUT_EVENT_ACK_STATE_CONSUMED) {
> On 2015/01/30 00:12:56, jdduke wrote:
> > Nit: Looks like we can remove this one-line if brace.
> 
> Done.
> 
>
https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
> File content/browser/renderer_host/input/touch_event_queue_unittest.cc
(right):
> 
>
https://codereview.chromium.org/886563005/diff/80001/content/browser/renderer...
> content/browser/renderer_host/input/touch_event_queue_unittest.cc:81:
> TouchEventQueue::Config config;
> On 2015/01/30 00:12:56, jdduke wrote:
> > Let's just remove this function entirely.
> 
> Done.
> 
>
https://codereview.chromium.org/886563005/diff/80001/tools/metrics/histograms...
> File tools/metrics/histograms/histograms.xml (left):
> 
>
https://codereview.chromium.org/886563005/diff/80001/tools/metrics/histograms...
> tools/metrics/histograms/histograms.xml:50609: -  <int value="-1619757314"
> label="touch-scrolling-mode"/>
> On 2015/01/30 15:17:28, asvitkine (slow this week) wrote:
> > Entries shouldn't be removed from histograms.xml, since they help decode
data
> > from older versions. You can revert the changes to this file.
> 
> Done.

LGTM.

[Alexei Svitkine (slow), 2015-02-03 21:13:53]

lgtm

[Avi (use Gerrit), 2015-02-03 21:45:06]

lgtm

kill kill kill kill

[lanwei, 2015-02-04 01:02:59]

The CQ bit was checked by lanwei@chromium.org

[commit-bot: I haz the power, 2015-02-04 01:04:03]

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

[commit-bot: I haz the power, 2015-02-04 01:09:19]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2015-02-04 01:10:35]

Patchset 3 (id:??) landed as
https://crrev.com/220483d0c533c2fa938fd5b150ab03ff77b195a7
Cr-Commit-Position: refs/heads/master@{#314456}