Merge "Make clock in QS not clickable" into sc-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
d031c6a3ce
@@ -16,9 +16,7 @@
|
|||||||
|
|
||||||
package com.android.systemui.qs;
|
package com.android.systemui.qs;
|
||||||
|
|
||||||
import android.content.Intent;
|
|
||||||
import android.os.Bundle;
|
import android.os.Bundle;
|
||||||
import android.provider.AlarmClock;
|
|
||||||
import android.view.View;
|
import android.view.View;
|
||||||
import android.view.View.OnClickListener;
|
import android.view.View.OnClickListener;
|
||||||
|
|
||||||
@@ -113,10 +111,7 @@ class QuickStatusBarHeaderController extends ViewController<QuickStatusBarHeader
|
|||||||
private View.OnClickListener mOnClickListener = new OnClickListener() {
|
private View.OnClickListener mOnClickListener = new OnClickListener() {
|
||||||
@Override
|
@Override
|
||||||
public void onClick(View v) {
|
public void onClick(View v) {
|
||||||
if (v == mClockView) {
|
if (v == mPrivacyChip) {
|
||||||
mActivityStarter.postStartActivityDismissingKeyguard(new Intent(
|
|
||||||
AlarmClock.ACTION_SHOW_ALARMS), 0);
|
|
||||||
} else if (v == mPrivacyChip) {
|
|
||||||
// If the privacy chip is visible, it means there were some indicators
|
// If the privacy chip is visible, it means there were some indicators
|
||||||
mUiEventLogger.log(PrivacyChipEvent.ONGOING_INDICATORS_CHIP_CLICK);
|
mUiEventLogger.log(PrivacyChipEvent.ONGOING_INDICATORS_CHIP_CLICK);
|
||||||
mPrivacyDialogController.showDialog(getContext());
|
mPrivacyDialogController.showDialog(getContext());
|
||||||
@@ -167,7 +162,6 @@ class QuickStatusBarHeaderController extends ViewController<QuickStatusBarHeader
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
protected void onViewAttached() {
|
protected void onViewAttached() {
|
||||||
mClockView.setOnClickListener(mOnClickListener);
|
|
||||||
mPrivacyChip.setOnClickListener(mOnClickListener);
|
mPrivacyChip.setOnClickListener(mOnClickListener);
|
||||||
|
|
||||||
mMicCameraIndicatorsEnabled = mPrivacyItemController.getMicCameraAvailable();
|
mMicCameraIndicatorsEnabled = mPrivacyItemController.getMicCameraAvailable();
|
||||||
@@ -188,7 +182,6 @@ class QuickStatusBarHeaderController extends ViewController<QuickStatusBarHeader
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
protected void onViewDetached() {
|
protected void onViewDetached() {
|
||||||
mClockView.setOnClickListener(null);
|
|
||||||
mColorExtractor.removeOnColorsChangedListener(mOnColorsChangedListener);
|
mColorExtractor.removeOnColorsChangedListener(mOnColorsChangedListener);
|
||||||
mPrivacyChip.setOnClickListener(null);
|
mPrivacyChip.setOnClickListener(null);
|
||||||
mStatusBarIconController.removeIconGroup(mIconManager);
|
mStatusBarIconController.removeIconGroup(mIconManager);
|
||||||
|
|||||||
@@ -126,6 +126,11 @@ class QuickStatusBarHeaderControllerTest : SysuiTestCase() {
|
|||||||
controller.onViewDetached()
|
controller.onViewDetached()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun testClockNotClickable() {
|
||||||
|
assertThat(clock.isClickable).isFalse()
|
||||||
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun testIgnoredSlotsOnAttached_noIndicators() {
|
fun testIgnoredSlotsOnAttached_noIndicators() {
|
||||||
setPrivacyController(micCamera = false, location = false)
|
setPrivacyController(micCamera = false, location = false)
|
||||||
|
|||||||
Reference in New Issue
Block a user