Commit 0cc54857 authored by Ian Vollick's avatar Ian Vollick Committed by Commit Bot

Assert that we don't use unsupported dialog properties

Since we don't use FILTER_TOUCH_FOR_SECURITY in touchless
mode, we should assert if we see it in our dialog properties.

Bug: None
Change-Id: I2f2f2f355c94775778822c8f92112260543e6fa8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1677245
Auto-Submit: Ian Vollick <vollick@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: default avatarMichael Thiessen <mthiesse@chromium.org>
Reviewed-by: default avatarBecky Zhou <huayinz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#672516}
parent 5dace10e
......@@ -135,6 +135,8 @@ public class TouchlessDialogPresenter extends Presenter {
PropertyModel model, Pair<ViewGroup, ModelListAdapter> view, PropertyKey propertyKey) {
ViewGroup dialogView = view.first;
ModelListAdapter optionsAdapter = view.second;
assert !model.get(ModalDialogProperties.FILTER_TOUCH_FOR_SECURITY);
assert propertyKey != ModalDialogProperties.FILTER_TOUCH_FOR_SECURITY;
// TODO(mdjones): If the default buttons are used assert no list items and convert the
// buttons to list items.
if (TouchlessDialogProperties.IS_FULLSCREEN == propertyKey) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment