Commit 46ce8535 authored by Mehran Mahmoudi's avatar Mehran Mahmoudi Committed by Commit Bot

Add calls to TouchlessUiController from TouchlessDialogPresenter

This calls TouchlessUiController::addModelToQueue and
TouchlessUicontroller::removeModelFromQueue when a dialog is shown
and dismissed, respectively. It also renames
AppHooks::createTouchlessMenuCoordinator()

Bug: 944512
Change-Id: I1f8b6ea6c2f411e04b2c24d4022399b05930527c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1564558Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Reviewed-by: default avatarMatthew Jones <mdjones@chromium.org>
Commit-Queue: Mehran Mahmoudi <mahmoudi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#650444}
parent 1755e48e
......@@ -379,7 +379,7 @@ public abstract class AppHooks {
/**
* @param activity An activity for access to different features.
*/
public void createTouchlessMenuCoordinator(ChromeActivity activity) {}
public void attachTouchlessMenuCoordinator(ChromeActivity activity) {}
/**
* @param activity An activity for access to different features.
......
......@@ -139,7 +139,7 @@ public class NoTouchActivity extends SingleTabActivity {
getFullscreenManager().setTab(getActivityTab());
mUiController = AppHooks.get().createTouchlessUiController(this);
AppHooks.get().createTouchlessMenuCoordinator(this);
AppHooks.get().attachTouchlessMenuCoordinator(this);
super.finishNativeInitialization();
}
......
......@@ -64,6 +64,12 @@ public class TouchlessDialogPresenter extends Presenter {
mDialog = new Dialog(mActivity, R.style.Theme_Chromium_DialogWhenLarge);
mDialog.setOnCancelListener(dialogInterface
-> dismissCurrentDialog(DialogDismissalCause.NAVIGATE_BACK_OR_TOUCH_OUTSIDE));
mDialog.setOnShowListener(dialog
-> AppHooks.get().getTouchlessUiControllerForActivity((ChromeActivity) mActivity)
.addModelToQueue(model));
mDialog.setOnDismissListener(dialog
-> AppHooks.get().getTouchlessUiControllerForActivity((ChromeActivity) mActivity)
.removeModelFromQueue(model));
// Cancel on touch outside should be disabled by default. The ModelChangeProcessor wouldn't
// notify change if the property is not set during initialization.
mDialog.setCanceledOnTouchOutside(false);
......
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