IssueTracker

Our 7.1.1 powered phone fails this test with dpi set to 640, but it will pass if dpi is changed to 560. We found it is same with Google Pixel, only its factory default dpi is 560 (when we change its dpi to 640, Pixel fails too). Our phone and  Pixel have the same failure info as below under 640 dpi,

02-22 13:41:11.179 11040 11059 D SamplePoint: Expected : ffdbdbdb at position x = 25 y = 65 , tested color : ffededed
02-22 13:41:11.180 11040 11059 D SamplePoint: Expected : ffdbdbdb at position x = 64 y = 65 , tested color : ffededed
02-22 13:41:11.181 11040 11059 D BitmapDumper: Saving file : testShadowLayout_capture.png in directory : ShadowTests
02-22 13:41:11.190 11040 11059 D BitmapDumper: Saving file : testShadowLayout_verifier_capture.png in directory : ShadowTests
02-22 13:41:11.227 11040 11040 D LifecycleMonitor: Lifecycle status change: android.uirendering.cts.testinfrastructure.DrawActivity@bdff6d8 in: PAUSED
02-22 13:41:11.228 11040 11054 I TestRunner: failed: testShadowLayout(android.uirendering.cts.testclasses.ShadowTests)
02-22 13:41:11.228 11040 11054 I TestRunner: ----- begin exception -----
02-22 13:41:11.229 11040 11054 I TestRunner: java.lang.AssertionError: Layout resource : android.uirendering.cts:layout/simple_shadow_layout
02-22 13:41:11.229 11040 11054 I TestRunner: Test ran in hardware
02-22 13:41:11.229 11040 11054 I TestRunner:
02-22 13:41:11.229 11040 11054 I TestRunner:    at org.junit.Assert.fail(Assert.java:88)
02-22 13:41:11.229 11040 11054 I TestRunner:    at org.junit.Assert.assertTrue(Assert.java:41)
02-22 13:41:11.229 11040 11054 I TestRunner:    at android.uirendering.cts.testinfrastructure.ActivityTestBase.assertBitmapIsVerified(ActivityTestBase.java:201)
02-22 13:41:11.229 11040 11054 I TestRunner:    at android.uirendering.cts.testinfrastructure.ActivityTestBase$TestCaseBuilder.runWithVerifier(ActivityTestBase.java:248)
02-22 13:41:11.229 11040 11054 I TestRunner:    at android.uirendering.cts.testclasses.ShadowTests.testShadowLayout(ShadowTests.java:70)
02-22 13:41:11.229 11040 11054 I TestRunner:    at java.lang.reflect.Method.invoke(Native Method)
02-22 13:41:11.229 11040 11054 I TestRunner:    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
02-22 13:41:11.229 11040 11054 I TestRunner:    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)

We believe the tolerance value in testShadowLayout case which is 48 now is not sufficient for phone with dpi set as 640, it should be at least 64 considering the
bitmap verification result. We also ask Qcom for help and they hold the same opinion and sugguest us to create an issue ticket here. Thank you for your time.

 
public void testShadowLayout() {
46        int shadowColorValue = 0xDB;
47        // Android TV theme overrides shadow opacity to be darker.
48        if (getActivity().getOnTv()) {
49            shadowColorValue = 0xBB;
50        }
51
52        // Use a higher threshold than default value (20), since we also double check gray scale;
53        GrayScaleVerifier verifier = new GrayScaleVerifier(
54                new Point[] {
55                        // view area
56                        new Point(25, 64),
57                        new Point(64, 64),
58                        // shadow area
59                        new Point(25, 65),
60                        new Point(64, 65)
61                },
62                new int[] {
63                        Color.WHITE,
64                        Color.WHITE,
65                        Color.rgb(shadowColorValue, shadowColorValue, shadowColorValue),
66                        Color.rgb(shadowColorValue, shadowColorValue, shadowColorValue),
67                },
68                48);
69
70        createTest()
71                .addLayout(R.layout.simple_shadow_layout, null, true/* HW only */)
72                .runWithVerifier(verifier);
73    }

PS: We find a more or less the similar issue like us, Issue 36949180, but it is tested against cts 6.0-r1. So we decide to open a new thread. Thank you for your time.