Add CompositorTouchAction to RuntimeEnabledFeatures.

Add CompositorTouchAction to RuntimeEnabledFeatures.

We are now working on handling touch actions in cc, adding the
runtime enabled features would make it possible for the two
logics (main thread only and both main thread and cc) to co-exist.

The flag will also be useful when we lauch the feature in the
future.

BUG=727839
Review-Url: https://codereview.chromium.org/2930493002
Cr-Commit-Position: refs/heads/master@{#478026}
Committed: https://chromium.googlesource.com/chromium/src/+/4d6bb36e3fcfb9742b6428e43a8e5df12681d0d0

[sunxd, 2017-06-06 17:47:49]

The CQ bit was checked by sunxd@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-06 17:48:19]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sunxd, 2017-06-06 17:49:46]

Description was changed from

==========
Add CompositorTouchAction to RuntimeEnabledFeatures.

BUG=727839
==========

to

==========
Add CompositorTouchAction to RuntimeEnabledFeatures.

BUG=727839
==========

[sunxd, 2017-06-06 17:49:46]

sunxd@chromium.org changed reviewers:
+ xidachen@chromium.org

[sunxd, 2017-06-06 17:50:40]

Hi Xida,

I tried adding the flag with reference to
https://www.chromium.org/blink/runtime-enabled-features.

But not sure if changes other than json5 are needed.

[xidachen, 2017-06-06 18:23:21]

On 2017/06/06 17:50:40, sunxd wrote:
> Hi Xida,
> 
> I tried adding the flag with reference to
> https://www.chromium.org/blink/runtime-enabled-features.
> 
> But not sure if changes other than json5 are needed.

LGTM.

I don't think we need to change any thing in about_flag*, so this should be
good. Please pass it to owners for review.

[sunxd, 2017-06-06 18:25:45]

Description was changed from

==========
Add CompositorTouchAction to RuntimeEnabledFeatures.

BUG=727839
==========

to

==========
Add CompositorTouchAction to RuntimeEnabledFeatures.

We are now working on handling touch actions in cc, adding the
runtime enabled features would make it possible for the two
logics (main thread only and both main thread and cc) to co-exist.

The flag will also be useful when we lauch the feature in the
future.

BUG=727839
==========

[sunxd, 2017-06-06 18:25:45]

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

[sunxd, 2017-06-06 18:27:04]

Hi flackr@,

Can you take a look at this patch before I ask the owner to review it?

Thanks!

[flackr, 2017-06-07 18:22:58]

lgtm

[sunxd, 2017-06-07 18:56:29]

sunxd@chromium.org changed reviewers:
+ jochen@chromium.org

[sunxd, 2017-06-07 18:57:41]

sunxd@chromium.org changed reviewers:
+ kinuko@chromium.org
- jochen@chromium.org

[sunxd, 2017-06-07 18:59:43]

Hi kinuko@?

Can you please review this patch? We want to implement handling touch actions in
impl side, and this will be the flag to turn on/off the feature.

Thanks.

[kinuko, 2017-06-08 04:56:08]

lgtm

[kinuko, 2017-06-08 04:56:12]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2017-06-08 04:56:28]

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-06-08 05:06:29]

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

[commit-bot: I haz the power, 2017-06-08 05:06:31]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[sunxd, 2017-06-08 15:45:39]

The CQ bit was checked by sunxd@chromium.org

[sunxd, 2017-06-08 15:45:40]

The patchset sent to the CQ was uploaded after l-g-t-m from flackr@chromium.org,
xidachen@chromium.org, kinuko@chromium.org
Link to the patchset: https://codereview.chromium.org/2930493002/#ps40001
(title: "Rebase - all the set functions are now capitalized")

[commit-bot: I haz the power, 2017-06-08 15:45:57]

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-06-08 18:17:08]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496936739642900,
"parent_rev": "2970fc91cb6d88be096b9974bef50c0fd2c2d331", "commit_rev":
"4d6bb36e3fcfb9742b6428e43a8e5df12681d0d0"}

[commit-bot: I haz the power, 2017-06-08 18:17:22]

Description was changed from

==========
Add CompositorTouchAction to RuntimeEnabledFeatures.

We are now working on handling touch actions in cc, adding the
runtime enabled features would make it possible for the two
logics (main thread only and both main thread and cc) to co-exist.

The flag will also be useful when we lauch the feature in the
future.

BUG=727839
==========

to

==========
Add CompositorTouchAction to RuntimeEnabledFeatures.

We are now working on handling touch actions in cc, adding the
runtime enabled features would make it possible for the two
logics (main thread only and both main thread and cc) to co-exist.

The flag will also be useful when we lauch the feature in the
future.

BUG=727839

Review-Url: https://codereview.chromium.org/2930493002
Cr-Commit-Position: refs/heads/master@{#478026}
Committed:
https://chromium.googlesource.com/chromium/src/+/4d6bb36e3fcfb9742b6428e43a8e...
==========

[commit-bot: I haz the power, 2017-06-08 18:17:24]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4d6bb36e3fcfb9742b6428e43a8e...