Move Touch Feedback flag to ui_base_switches

Move Touch Feedback flag to ui_base_switches

Move flag for touch feedback experiments to ui_base_switches from ash_switches. Update the classes currently using the ash switch.

This will allow for the experiments which are on components under ui/

BUG=398398
Committed: https://crrev.com/37f21b8e03b0647492f24606cbd87ba0647374e9
Cr-Commit-Position: refs/heads/master@{#297246}

[jonross, 2014-09-17 21:27:01]

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

[jonross, 2014-09-17 21:27:02]

I plan to land the move of the switch, before updating all the CLs currently
using the ash variant.

[flackr, 2014-09-17 21:34:45]

LGTM. Looks much cleaner, thanks for moving tracking the result into a helper
function! Might be worth mentioning the move to ui/base is needed for ui/
components which are going to update based on the flag as well.

[jonross, 2014-09-17 21:40:20]

jonross@chromium.org changed reviewers:
+ skuhne@chromium.org

[jonross, 2014-09-17 21:41:21]

Hi Stefan,

Could you provide an owner review for the changes to ash/shelf and
ash/system/tray?

Small refactor of the switch we are using for touch experiments

[jonross, 2014-09-17 21:45:00]

jonross@chromium.org changed reviewers:
+ ben@chromium.org

[jonross, 2014-09-17 21:45:00]

Hi Ben,

I'm working on some touch feedback experiments. We want to move our current
switch from ash_switches to ui_base_switches in order to account for work being
done on some ui/ views.

Could you provide an owner review for the files under ui/base?

Thanks

[jonross, 2014-09-17 21:46:20]

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

[jonross, 2014-09-17 21:46:21]

Hi Alexei,

Could you provide an owner review to tools/metrics/histograms/histograms.xml.
I've updated it to account for moving a switch from ash_switches to
ui_base_switches.

Thanks

[Mr4D (OOO till 08-26), 2014-09-18 23:27:34]

lgtm for ash/system/tray

[Alexei Svitkine (slow), 2014-09-19 04:17:30]

https://codereview.chromium.org/580903003/diff/1/tools/metrics/histograms/his...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/580903003/diff/1/tools/metrics/histograms/his...
tools/metrics/histograms/histograms.xml:44612: -  <int value="1163255347"
label="ash-enable-touch-view-touch-feedback"/>
Please don't remove obsolete entries as they're still needed for decoding data
from older builds. Just add the new one but keep this one in.

[jonross, 2014-09-19 13:41:51]

https://codereview.chromium.org/580903003/diff/1/tools/metrics/histograms/his...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/580903003/diff/1/tools/metrics/histograms/his...
tools/metrics/histograms/histograms.xml:44612: -  <int value="1163255347"
label="ash-enable-touch-view-touch-feedback"/>
On 2014/09/19 04:17:30, Alexei Svitkine wrote:
> Please don't remove obsolete entries as they're still needed for decoding data
> from older builds. Just add the new one but keep this one in.

Done.

[Alexei Svitkine (slow), 2014-09-19 14:08:05]

lgtm

[jonross, 2014-09-25 14:48:31]

Hi Ben,

Would you be able to provide an owner's review for my changes to ui/base/.

Thanks,
Jon

[jonross, 2014-09-29 15:03:10]

jonross@chromium.org changed reviewers:
- ben@chromium.org

[jonross, 2014-09-29 15:03:51]

jonross@chromium.org changed reviewers:
+ sky@chromium.org

[jonross, 2014-09-29 15:03:52]

sky@chromium.org: 

I'm working on some touch feedback experiments. We want to move our current
switch from ash_switches to ui_base_switches in order to account for work being
done on some ui/ views.

Could you provide an owner review for the files under ui/base?

Thanks

[sky, 2014-09-29 18:05:54]

ui LGTM

[jonross, 2014-09-29 18:09:39]

The CQ bit was checked by jonross@chromium.org

[commit-bot: I haz the power, 2014-09-29 18:10:23]

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

[commit-bot: I haz the power, 2014-09-29 18:28:34]

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

[commit-bot: I haz the power, 2014-09-29 18:28:35]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jonross, 2014-09-29 19:05:58]

The CQ bit was checked by jonross@chromium.org

[commit-bot: I haz the power, 2014-09-29 19:06:54]

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

[commit-bot: I haz the power, 2014-09-29 20:31:43]

Committed patchset #3 (id:40001) as d42577806f2d8f7a5828f9e3794dd6af78d01e0c

[commit-bot: I haz the power, 2014-09-29 20:32:23]

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