Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e3a095e1 authored by John Reck's avatar John Reck Committed by Android (Google) Code Review
Browse files

Merge changes from topic "revert-30679429-forcedark-colorarea-detection-KGFLTNVUGM" into main 

* changes:
  Revert "feat(force invert): detect the polarity of an app to dec..."
  Revert "feat(force invert): force invert the entire app if its p..."
  Revert "feat(force invert): also count areas of draw calls with ..."
parents d68a4f66 306c8de5

core/java/android/view/ViewRootImpl.java

+16 −5
Original line number Diff line number Diff line
@@ -136,10 +136,10 @@ import static com.android.internal.annotations.VisibleForTesting.Visibility.PACK 
import static com.android.text.flags.Flags.disableHandwritingInitiatorForIme;

import static com.android.window.flags.Flags.enableBufferTransformHintFromDisplay;

import static com.android.window.flags.Flags.enableWindowContextResourcesUpdateOnConfigChange;

import static com.android.window.flags.Flags.fixViewRootCallTrace;

import static com.android.window.flags.Flags.predictiveBackSwipeEdgeNoneApi;

import static com.android.window.flags.Flags.reduceChangedExclusionRectsMsgs;

import static com.android.window.flags.Flags.setScPropertiesInClient;

import static com.android.window.flags.Flags.fixViewRootCallTrace;















import android.Manifest;















import android.accessibilityservice.AccessibilityService;










@@ -191,6 +191,7 @@ import android.graphics.Rect;













import android.graphics.RectF;















import android.graphics.Region;















import android.graphics.RenderNode;













import android.graphics.drawable.ColorDrawable;















import android.graphics.drawable.Drawable;















import android.graphics.drawable.GradientDrawable;















import android.hardware.SyncFence;










@@ -294,6 +295,7 @@ import com.android.internal.os.SomeArgs;













import com.android.internal.policy.DecorView;















import com.android.internal.policy.PhoneFallbackEventHandler;















import com.android.internal.protolog.ProtoLog;













import com.android.internal.util.ContrastColorUtil;















import com.android.internal.util.FastPrintWriter;















import com.android.internal.view.BaseSurfaceHolder;















import com.android.internal.view.RootViewSurfaceTaker;










@@ -2087,12 +2089,21 @@ public final class ViewRootImpl implements ViewParent,













                // preference for dark mode in configuration.uiMode. Instead, we assume that both















                // force invert and the system's dark theme are enabled.















                if (shouldApplyForceInvertDark()) {













                    // We will use HWUI color area detection to determine if it should actually be













                    // inverted. Checking light theme simply gives the developer a way to "opt-out"













                    // of force invert.













                    // TODO: b/368725782 - Use hwui color area detection instead of / in













                    //  addition to these heuristics.















                    final boolean isLightTheme =















                            a.getBoolean(R.styleable.Theme_isLightTheme, false);













                    if (isLightTheme) {













                    final boolean isBackgroundColorLight;













                    if (mView != null && mView.getBackground()













                            instanceof ColorDrawable colorDrawable) {













                        isBackgroundColorLight =













                                !ContrastColorUtil.isColorDarkLab(colorDrawable.getColor());













                    } else {













                        // Treat unknown as light, so that only isLightTheme is used to determine













                        // force dark treatment.













                        isBackgroundColorLight = true;













                    }













                    if (isLightTheme && isBackgroundColorLight) {















                        return ForceDarkType.FORCE_INVERT_COLOR_DARK;















                    } else {















                        return ForceDarkType.NONE;

































core/tests/coretests/src/android/view/ViewRootImplTest.java



+2
−3




























Original line number
Diff line number
Diff line








@@ -1548,7 +1548,7 @@ public class ViewRootImplTest {





























    @Test















    @EnableFlags(FLAG_FORCE_INVERT_COLOR)













    public void determineForceDarkType_isLightThemeAndNotLightBackground_returnsForceInvertColorDark()













    public void determineForceDarkType_isLightThemeAndNotLightBackground_returnsNone()















            throws Exception {















        // Set up configurations for force invert color















        waitForSystemNightModeActivated(true);









@@ -1557,8 +1557,7 @@ public class ViewRootImplTest {













        setUpViewAttributes(/* isLightTheme= */ true, /* isLightBackground = */ false);































        TestUtils.waitUntil("Waiting for ForceDarkType to be ready",













                () -> (mViewRootImpl.determineForceDarkType()













                        == ForceDarkType.FORCE_INVERT_COLOR_DARK));













                () -> (mViewRootImpl.determineForceDarkType() == ForceDarkType.NONE));















    }































    @Test

































libs/hwui/Android.bp



+0
−1




























Original line number
Diff line number
Diff line








@@ -616,7 +616,6 @@ cc_defaults {













        "Animator.cpp",















        "AnimatorManager.cpp",















        "CanvasTransform.cpp",













        "ColorArea.cpp",















        "DamageAccumulator.cpp",















        "DeviceInfo.cpp",















        "FrameInfo.cpp",

































libs/hwui/CanvasTransform.cpp



+1
−1




























Original line number
Diff line number
Diff line








@@ -127,7 +127,7 @@ static BitmapPalette paletteForColorHSV(SkColor color) {













    return hsv[2] >= .5f ? BitmapPalette::Light : BitmapPalette::Dark;















}





























BitmapPalette filterPalette(const SkPaint* paint, BitmapPalette palette) {













static BitmapPalette filterPalette(const SkPaint* paint, BitmapPalette palette) {















    if (palette == BitmapPalette::Unknown || !paint || !paint->getColorFilter()) {















        return palette;















    }

































libs/hwui/CanvasTransform.h



+0
−3




























Original line number
Diff line number
Diff line








@@ -48,7 +48,4 @@ bool transformPaint(ColorTransform transform, SkPaint* paint, BitmapPalette pale













SkColor transformColor(ColorTransform transform, SkColor color);















SkColor transformColorInverse(ColorTransform transform, SkColor color);





























/** Returns a palette corrected in case it is tinted by the given paint's filter */













BitmapPalette filterPalette(const SkPaint* paint, BitmapPalette palette);





























}  // namespace android::uirenderer