New issue
Advanced search Search tips

Issue 701135 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug


Participants' hotlists:
Fixing-touch


Sign in to add a comment

Add a flag to allow easy selection of tablet mode.

Project Member Reported by zork@chromium.org, Mar 13 2017

Issue description

We should add an option on chrome://flags to force touchview vs clamshell vs auto.

When set, it should enable all the right things for the selected UI type, such as keyboard, ash debug shortcut, accelerometer, etc.

auto should decide the mode based on the existence of keyboard/mouse and availability of touch.
 

Comment 1 by zork@chromium.org, Mar 13 2017

Owner: sammiequon@chromium.org
Status: Assigned (was: Untriaged)
Thanks zork!

Suggestion: instead of an enable/disable flag, perhaps make it a dropdown with 3 options:
- TouchView
- Clamshell
- Auto (default)


Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/df9bf7c32b4361a7b66a241317ca3571f58c28b3

commit df9bf7c32b4361a7b66a241317ca3571f58c28b3
Author: sammiequon <sammiequon@chromium.org>
Date: Wed Mar 29 15:17:03 2017

CrOS: Move force UI mode switch from switches to ash switches.

We will need this switch in ash switches to use with ash. See dependent patchset for usage.

TEST=unit_tests --gtest_filter="AboutFlagsHistogramTest.*"
BUG= 701135 

Review-Url: https://codereview.chromium.org/2765373002
Cr-Commit-Position: refs/heads/master@{#460391}

[modify] https://crrev.com/df9bf7c32b4361a7b66a241317ca3571f58c28b3/ash/common/ash_switches.cc
[modify] https://crrev.com/df9bf7c32b4361a7b66a241317ca3571f58c28b3/ash/common/ash_switches.h
[modify] https://crrev.com/df9bf7c32b4361a7b66a241317ca3571f58c28b3/chrome/browser/about_flags.cc
[modify] https://crrev.com/df9bf7c32b4361a7b66a241317ca3571f58c28b3/chrome/common/chrome_switches.cc
[modify] https://crrev.com/df9bf7c32b4361a7b66a241317ca3571f58c28b3/chrome/common/chrome_switches.h

Project Member

Comment 6 by bugdroid1@chromium.org, Mar 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1d6e7115c66667ae3f401de68bf7cfbc7e694f79

commit 1d6e7115c66667ae3f401de68bf7cfbc7e694f79
Author: sammiequon <sammiequon@chromium.org>
Date: Wed Mar 29 17:09:30 2017

CrOS: Apply flag which enables us to force maximize mode.

There is a flag which lets user choose if they want to force clamshell/touchView mode regardless of device orientation(s). This CL applys the functionality of that flag.

TEST=ash_unittests --gtest_filter="MaximizeModeControllerTest.Force*"
BUG= 701135 

Review-Url: https://codereview.chromium.org/2775683002
Cr-Commit-Position: refs/heads/master@{#460429}

[modify] https://crrev.com/1d6e7115c66667ae3f401de68bf7cfbc7e694f79/ash/common/wm/maximize_mode/maximize_mode_controller.cc
[modify] https://crrev.com/1d6e7115c66667ae3f401de68bf7cfbc7e694f79/ash/common/wm/maximize_mode/maximize_mode_controller.h
[modify] https://crrev.com/1d6e7115c66667ae3f401de68bf7cfbc7e694f79/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
9532.0.0, 60.0.3092.0

Sign in to add a comment