Merge "Update dark mode on notifications when content is updated" into lmp-mr1-dev

This commit is contained in:
Jorim Jaggi
2014-12-02 19:35:39 +00:00
committed by Android (Google) Code Review
4 changed files with 24 additions and 35 deletions

View File

@@ -244,6 +244,9 @@ public class NotificationContentView extends FrameLayout {
public void notifyContentUpdated() { public void notifyContentUpdated() {
selectLayout(false /* animate */, true /* force */); selectLayout(false /* animate */, true /* force */);
if (mContractedChild != null) {
mContractedWrapper.setDark(mDark, false /* animate */, 0 /* delay */);
}
} }
public boolean isContentExpandable() { public boolean isContentExpandable() {

View File

@@ -27,7 +27,6 @@ import com.android.systemui.statusbar.phone.NotificationPanelView;
public class NotificationCustomViewWrapper extends NotificationViewWrapper { public class NotificationCustomViewWrapper extends NotificationViewWrapper {
private final ViewInvertHelper mInvertHelper; private final ViewInvertHelper mInvertHelper;
private boolean mDark;
protected NotificationCustomViewWrapper(View view) { protected NotificationCustomViewWrapper(View view) {
super(view); super(view);
@@ -36,8 +35,6 @@ public class NotificationCustomViewWrapper extends NotificationViewWrapper {
@Override @Override
public void setDark(boolean dark, boolean fade, long delay) { public void setDark(boolean dark, boolean fade, long delay) {
if (mDark != dark) {
mDark = dark;
if (fade) { if (fade) {
mInvertHelper.fade(dark, delay); mInvertHelper.fade(dark, delay);
} else { } else {
@@ -45,4 +42,3 @@ public class NotificationCustomViewWrapper extends NotificationViewWrapper {
} }
} }
} }
}

View File

@@ -24,19 +24,14 @@ import android.view.View;
*/ */
public class NotificationMediaViewWrapper extends NotificationTemplateViewWrapper { public class NotificationMediaViewWrapper extends NotificationTemplateViewWrapper {
private boolean mDark;
protected NotificationMediaViewWrapper(Context ctx, View view) { protected NotificationMediaViewWrapper(Context ctx, View view) {
super(ctx, view); super(ctx, view);
} }
@Override @Override
public void setDark(boolean dark, boolean fade, long delay) { public void setDark(boolean dark, boolean fade, long delay) {
if (mDark != dark) {
mDark = dark;
// Only update the large icon, because the rest is already inverted. // Only update the large icon, because the rest is already inverted.
setPictureGrayscale(dark, fade, delay); setPictureGrayscale(dark, fade, delay);
} }
} }
}

View File

@@ -51,8 +51,6 @@ public class NotificationTemplateViewWrapper extends NotificationViewWrapper {
private final int mIconBackgroundDarkColor; private final int mIconBackgroundDarkColor;
private final Interpolator mLinearOutSlowInInterpolator; private final Interpolator mLinearOutSlowInInterpolator;
private boolean mDark;
protected NotificationTemplateViewWrapper(Context ctx, View view) { protected NotificationTemplateViewWrapper(Context ctx, View view) {
super(view); super(view);
mIconDarkAlpha = ctx.getResources().getInteger(R.integer.doze_small_icon_alpha); mIconDarkAlpha = ctx.getResources().getInteger(R.integer.doze_small_icon_alpha);
@@ -95,8 +93,6 @@ public class NotificationTemplateViewWrapper extends NotificationViewWrapper {
@Override @Override
public void setDark(boolean dark, boolean fade, long delay) { public void setDark(boolean dark, boolean fade, long delay) {
if (mDark != dark) {
mDark = dark;
if (mInvertHelper != null) { if (mInvertHelper != null) {
if (fade) { if (fade) {
mInvertHelper.fade(dark, delay); mInvertHelper.fade(dark, delay);
@@ -115,7 +111,6 @@ public class NotificationTemplateViewWrapper extends NotificationViewWrapper {
} }
setPictureGrayscale(dark, fade, delay); setPictureGrayscale(dark, fade, delay);
} }
}
protected void setPictureGrayscale(boolean grayscale, boolean fade, long delay) { protected void setPictureGrayscale(boolean grayscale, boolean fade, long delay) {
if (mPicture != null) { if (mPicture != null) {