cros: Implement improved stylus mode exiting

Desktop / Chromium - Sammie Quon [google.com] - 19 July 2017 22:45 EDT

This CL does the following: 1) If we choose a tool that is a "mode", the palette tray stays active. 2) If a tool that is a "mode" is active, tapping the palette tray deactives the tool. 3) Remove the checkmarks (because they can't be seen anymore with #2). 4) Add a seperator between tools that are "actions" and tools that are "modes".

Test: ash_unittests --gtest_filter="PaletteTrayTest.*" Bug: 742524 Change-Id: I70d6e07a460250e7a22059973de036146dd753a6 Reviewed-on: https://chromium-review.googlesource.com/572066 Commit-Queue: Sammie Quon

7645a80 cros: Implement improved stylus mode exiting.
ash/BUILD.gn | 1 +
ash/system/palette/common_palette_tool.cc | 22 ++----
ash/system/palette/palette_tool_manager.cc | 15 ++--
ash/system/palette/palette_tool_manager.h | 4 +-
ash/system/palette/palette_tray.cc | 58 +++++++++++++---
ash/system/palette/palette_tray.h | 22 +++++-
ash/system/palette/palette_tray_unittest.cc | 104 ++++++++++++++++++++++++++++
7 files changed, 192 insertions(+), 34 deletions(-)

Upstream: git.chromium.org


  • Share