summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorpowei@chromium.org <powei@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-27 19:31:48 +0000
committerpowei@chromium.org <powei@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-27 19:31:48 +0000
commitec3b0785155d9967e7e115108b0030590b77e496 (patch)
treee51f93501cdcced5d4de9941c43638fee91961c7
parent1c2bbf47bd5c796939967934f23862b2ef5a4814 (diff)
downloadchromium_src-ec3b0785155d9967e7e115108b0030590b77e496.zip
chromium_src-ec3b0785155d9967e7e115108b0030590b77e496.tar.gz
chromium_src-ec3b0785155d9967e7e115108b0030590b77e496.tar.bz2
android: Add test for ContentViewCore readback
This test renders a blue screen and grabs a snapshot of the content. The test checks that the pixel in the middle of the snapshot is blue. BUG=372792 NOTRY=true Review URL: https://codereview.chromium.org/358513002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280381 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat
-rw-r--r--build/android/findbugs_filter/findbugs_known_bugs.txt2
-rw-r--r--content/public/android/javatests/src/org/chromium/content/browser/ContentViewReadbackTest.java124
2 files changed, 126 insertions, 0 deletions
diff --git a/build/android/findbugs_filter/findbugs_known_bugs.txt b/build/android/findbugs_filter/findbugs_known_bugs.txt
index 35a6047..ed860ac 100644
--- a/build/android/findbugs_filter/findbugs_known_bugs.txt
+++ b/build/android/findbugs_filter/findbugs_known_bugs.txt
@@ -25,3 +25,5 @@ M V EI: org.chromium.content.browser.LoadUrlParams.getPostData() may expose inte
M M LI: Incorrect lazy initialization of static field org.chromium.chrome.browser.sync.ProfileSyncService.sSyncSetupManager in org.chromium.chrome.browser.sync.ProfileSyncService.get(Context) At ProfileSyncService.java
M V EI: org.chromium.android_webview.AwWebResourceResponse.getResponseHeaderNames() may expose internal representation by returning AwWebResourceResponse.mResponseHeaderNames At AwWebResourceResponse.java
M V EI: org.chromium.android_webview.AwWebResourceResponse.getResponseHeaderValues() may expose internal representation by returning AwWebResourceResponse.mResponseHeaderValues At AwWebResourceResponse.java
+M C NP: Null passed for nonnull parameter of ContentReadbackHandler$GetBitmapCallback.onFinishGetBitmap(boolean, Bitmap) in org.chromium.content.browser.ContentReadbackHandler.getCompositorBitmapAsync(WindowAndroid, ContentReadbackHandler$GetBitmapCallback) At ContentReadbackHandler.java
+M C NP: Null passed for nonnull parameter of ContentReadbackHandler$GetBitmapCallback.onFinishGetBitmap(boolean, Bitmap) in org.chromium.content.browser.ContentReadbackHandler.getContentBitmapAsync(float, Rect, ContentViewCore, ContentReadbackHandler$GetBitmapCallback) At ContentReadbackHandler.java
diff --git a/content/public/android/javatests/src/org/chromium/content/browser/ContentViewReadbackTest.java b/content/public/android/javatests/src/org/chromium/content/browser/ContentViewReadbackTest.java
new file mode 100644
index 0000000..79049bf
--- /dev/null
+++ b/content/public/android/javatests/src/org/chromium/content/browser/ContentViewReadbackTest.java
@@ -0,0 +1,124 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.content.browser;
+
+import static org.chromium.base.test.util.ScalableTimeout.scaleTimeout;
+
+import android.graphics.Bitmap;
+import android.graphics.Color;
+import android.graphics.Rect;
+import android.os.SystemClock;
+import android.test.suitebuilder.annotation.SmallTest;
+
+import junit.framework.AssertionFailedError;
+
+import org.chromium.base.ThreadUtils;
+import org.chromium.base.test.util.Feature;
+import org.chromium.base.test.util.UrlUtils;
+import org.chromium.content.browser.ContentReadbackHandler.GetBitmapCallback;
+import org.chromium.content.browser.test.util.Criteria;
+import org.chromium.content.browser.test.util.CriteriaHelper;
+import org.chromium.content_shell_apk.ContentShellTestBase;
+
+import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.concurrent.atomic.AtomicInteger;
+
+/**
+ * Tests that we can readback the contents of a ContentView.
+ */
+public class ContentViewReadbackTest extends ContentShellTestBase {
+
+ private static final int COLOR_THRESHOLD = 8;
+ private static final long WAIT_FOR_READBACK_TIMEOUT = scaleTimeout(10000);
+
+ @Override
+ protected void setUp() throws Exception {
+ super.setUp();
+ launchContentShellWithUrl(UrlUtils.encodeHtmlDataUri(
+ "<html style=\"background: #00f;\"><head><style>body { height: 5000px; }</style>" +
+ "</head></html>"));
+ assertTrue("Page failed to load", waitForActiveShellToBeDoneLoading());
+ }
+
+ private static void assertEqualColor(int actual, int expected, int threshold) {
+ int deltaR = Math.abs(Color.red(actual) - Color.red(expected));
+ int deltaG = Math.abs(Color.green(actual) - Color.green(expected));
+ int deltaB = Math.abs(Color.blue(actual) - Color.blue(expected));
+ if (deltaR > threshold || deltaG > threshold || deltaB > threshold) {
+ throw new AssertionFailedError(
+ "Color does not match; expected " + expected + ", got " + actual);
+ }
+ }
+
+ private void assertWaitForYScroll(final int previousYScroll) throws InterruptedException {
+ assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
+ @Override
+ public boolean isSatisfied() {
+ return getContentViewCore().getNativeScrollYForTest() > previousYScroll;
+ }
+ }));
+ }
+
+ private void assertWaitForReadback(final AtomicBoolean readbackDone)
+ throws InterruptedException {
+ assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
+ @Override
+ public boolean isSatisfied() {
+ return readbackDone.get();
+ }
+ }, WAIT_FOR_READBACK_TIMEOUT, CriteriaHelper.DEFAULT_POLLING_INTERVAL));
+ }
+
+ /**
+ * Test to make sure the screenshots captured of the ContentView are not blank.
+ *
+ * @throws InterruptedException
+ */
+ @SmallTest
+ @Feature({"Main"})
+ public void testScreenshotIsNotBlank() throws Throwable {
+ final AtomicBoolean readbackDone = new AtomicBoolean(false);
+ final AtomicInteger color = new AtomicInteger();
+
+ // We wait on the fling to make sure that the compositor side has a layer available by the
+ // time we reach readback.
+ int previousYScroll = getContentViewCore().getNativeScrollYForTest();
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ getContentViewCore().flingForTest(SystemClock.uptimeMillis(), 0, 0, 0, -100);
+ }
+ });
+ assertWaitForYScroll(previousYScroll);
+
+ runTestOnUiThread(new Runnable() {
+ @Override
+ public void run() {
+ ContentReadbackHandler contentReadbackHandler = new ContentReadbackHandler() {
+ @Override
+ protected boolean readyForReadback() {
+ return true;
+ }
+ };
+
+ GetBitmapCallback callback = new GetBitmapCallback() {
+ @Override
+ public void onFinishGetBitmap(boolean success, Bitmap bitmap) {
+ // Verify a pixel in the center of the screenshot.
+ color.set(bitmap.getPixel(bitmap.getWidth() / 2, bitmap.getHeight() / 2));
+ readbackDone.set(success);
+ }
+ };
+
+ contentReadbackHandler.initNativeContentReadbackHandler();
+ contentReadbackHandler.getContentBitmapAsync(1.0f, new Rect(), getContentViewCore(),
+ callback);
+ }
+ });
+ assertWaitForReadback(readbackDone);
+
+ assertEqualColor(color.get(), Color.BLUE, COLOR_THRESHOLD);
+ }
+}
generated by cgit v1.1 at 2025-05-28 11:01:08 +0000