Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(376)

Issue 192763003: Enabled gesture tap highlights by default on all platforms. (Closed)

Created:
6 years, 9 months ago by bokan
Modified:
6 years, 9 months ago
Reviewers:
sky
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, nasko+codewatch_chromium.org, jam, creis+watch_chromium.org
Visibility:
Public.

Description

Enabled gesture tap highlights by default on all platforms. This was "enabled" a few months ago in r160367 but the flag remained off by default. This CL enables the experiment in about:flags for all platforms except Mac (since it has no touch support) and makes the "enabled" flag the default everywhere. BUG=263516 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=256102

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -6 lines) Patch
M chrome/browser/about_flags.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/render_view_host_impl.cc View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
bokan
6 years, 9 months ago (2014-03-10 19:25:47 UTC) #1
sky
LGTM
6 years, 9 months ago (2014-03-10 19:38:24 UTC) #2
bokan
The CQ bit was checked by [email protected]
6 years, 9 months ago (2014-03-10 19:40:29 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/192763003/20001
6 years, 9 months ago (2014-03-10 19:43:52 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/192763003/20001
6 years, 9 months ago (2014-03-10 21:41:53 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/192763003/20001
6 years, 9 months ago (2014-03-10 22:20:38 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/192763003/20001
6 years, 9 months ago (2014-03-10 23:39:39 UTC) #7
commit-bot: I haz the power
6 years, 9 months ago (2014-03-11 02:50:16 UTC) #8
Message was sent while issue was closed.
Change committed as 256102

Powered by Google App Engine
This is Rietveld 408576698