Added projection mode to touch HUD

Added projection mode to touch HUD

Projection mode is useful in presentations when the screen is projected
using a projector. In this mode, only active touch points are shown
without any trails.

BUG=233565
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196931

[mohsen, 2013-04-25 17:38:01]

Please take a look...

[sadrul, 2013-04-26 16:17:24]

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
File ash/touch/touch_observer_hud.cc (right):

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
ash/touch/touch_observer_hud.cc:395: virtual void AnimationEnded(const
ui::Animation* animation) {
OVERRIDE

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
ash/touch/touch_observer_hud.cc:410: for (int i = 0; i < kMaxPaths; ++i)
Call AnimationEnded(animation) instead?

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
ash/touch/touch_observer_hud.cc:418: const TouchLog* const touch_log_;
Can you use const TouchLog& instead?

[mohsen, 2013-04-26 17:22:03]

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
File ash/touch/touch_observer_hud.cc (right):

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
ash/touch/touch_observer_hud.cc:395: virtual void AnimationEnded(const
ui::Animation* animation) {
On 2013/04/26 16:17:24, sadrul wrote:
> OVERRIDE

Done.

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
ash/touch/touch_observer_hud.cc:410: for (int i = 0; i < kMaxPaths; ++i)
On 2013/04/26 16:17:24, sadrul wrote:
> Call AnimationEnded(animation) instead?

Done.

https://codereview.chromium.org/14399005/diff/4001/ash/touch/touch_observer_h...
ash/touch/touch_observer_hud.cc:418: const TouchLog* const touch_log_;
On 2013/04/26 16:17:24, sadrul wrote:
> Can you use const TouchLog& instead?

Yep. Done.

[sadrul, 2013-04-26 17:29:38]

LGTM

[mohsen, 2013-04-26 17:34:56]

sky@: for OWNERS in /ash/, please.

[sky, 2013-04-26 19:40:06]

Rubber stamp LGTM as Sadrul reviewed it.

[commit-bot: I haz the power, 2013-04-26 19:41:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/14399005/9003

[commit-bot: I haz the power, 2013-04-26 20:47:02]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win7_aura.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-04-26 21:42:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/14399005/15002

[commit-bot: I haz the power, 2013-04-26 21:53:46]

Retried try job too often on linux_chromeos for step(s) ash_unittests,
aura_unittests, base_unittests, browser_tests, cacheinvalidation_unittests,
check_deps, chromeos_unittests, components_unittests, content_browsertests,
content_unittests, crypto_unittests, dbus_unittests, device_unittests,
gpu_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
media_unittests, net_unittests, ppapi_unittests, printing_unittests,
sandbox_linux_unittests, sql_unittests, sync_unit_tests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-04-27 03:06:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/14399005/15002

[commit-bot: I haz the power, 2013-04-27 03:25:08]

Change committed as 196931