[WebView] Add test for visual state API when view dettached.

android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java

 // Copyright 2015 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.android_webview.test;
 
 import android.graphics.Bitmap;
 import android.graphics.Color;
+import android.graphics.Rect;
+import android.os.Build;
 import android.test.suitebuilder.annotation.SmallTest;
 import android.view.View;
 import android.webkit.WebChromeClient;
 
 import org.chromium.android_webview.AwContents;
 import org.chromium.android_webview.AwContents.VisualStateCallback;
 import org.chromium.android_webview.test.util.CommonResources;
 import org.chromium.android_webview.test.util.GraphicsTestUtils;
 import org.chromium.android_webview.test.util.JavascriptEventObserver;
 import org.chromium.base.test.util.Feature;
+import org.chromium.base.test.util.MinAndroidSdkLevel;
 import org.chromium.content.browser.ContentViewCore;
 import org.chromium.content.browser.test.util.CallbackHelper;
 import org.chromium.content.browser.test.util.DOMUtils;
 import org.chromium.content_public.browser.LoadUrlParams;
 
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicReference;
 
 /**
  * Visual state related tests.
  */
+@MinAndroidSdkLevel(Build.VERSION_CODES.KITKAT)
 public class VisualStateTest extends AwTestBase {
 
     private static final String WAIT_FOR_JS_TEST_URL =
             "file:///android_asset/visual_state_waits_for_js_test.html";
     private static final String FULLSCREEN_TEST_URL =
             "file:///android_asset/visual_state_during_fullscreen_test.html";
     private static final String UPDATE_COLOR_CONTROL_ID = "updateColorControl";
     private static final String ENTER_FULLSCREEN_CONTROL_ID = "enterFullscreenControl";
 
     private TestAwContentsClient mContentsClient = new TestAwContentsClient();
@@ skipping 273 matching lines @@
 
         loadUrlSync(awContents, awContentsClient.getOnPageFinishedHelper(), FULLSCREEN_TEST_URL);
 
         assertTrue(readyToEnterFullscreenSignal.await(
                 AwTestBase.WAIT_TIMEOUT_MS, TimeUnit.MILLISECONDS));
         DOMUtils.clickNode(VisualStateTest.this, contentViewCore, ENTER_FULLSCREEN_CONTROL_ID);
 
         assertTrue(testFinishedSignal.await(AwTestBase.WAIT_TIMEOUT_MS, TimeUnit.MILLISECONDS));
     }
 
+    @Feature({"AndroidWebView"})
+    @SmallTest
+    public void testVisualStateCallbackWhenContainerViewDetached()
+            throws Throwable {
+        final CountDownLatch readyToEnterFullscreenSignal = new CountDownLatch(1);
+        final CountDownLatch hasCustomViewSignal = new CountDownLatch(1);
+        final CountDownLatch testFinishedSignal = new CountDownLatch(1);
+
+        final AtomicReference<AwContents> awContentsRef = new AtomicReference<>();

[hush (inactive), 2015/03/04 18:39:08]

Why not "AtomicReference<AwContents>" on the right?
The comment for the next line.

[boliu, 2015/03/04 19:16:43]

http://docs.oracle.com/javase/7/docs/technotes/guides/language/type-inference...

[hush (inactive), 2015/03/04 19:39:54]

Okay. 

+        final AtomicReference<View> customViewRef = new AtomicReference<>();
+
+        final TestAwContentsClient awContentsClient = new TestAwContentsClient() {
+            @Override
+            public void onPageFinished(String url) {
+                super.onPageFinished(url);
+                readyToEnterFullscreenSignal.countDown();
+            }
+
+            @Override
+            public void onShowCustomView(
+                    final View customView, WebChromeClient.CustomViewCallback callback) {
+                // Please note that we don't attach the custom view to the window here
+                // (awContentsClient is an instance of TestAwContentsClient, not
+                // FullScreenVideoTestAwContentsClient).
+                customView.setClipBounds(new Rect(0, 0, 100, 100));
+                customView.measure(100, 100);
+                customView.layout(0, 0, 100, 100);
+                customViewRef.set(customView);
+                hasCustomViewSignal.countDown();
+            }
+        };
+        final AwTestContainerView testView = createAwTestContainerViewOnMainSync(awContentsClient);
+        final AwContents awContents = testView.getAwContents();
+        awContentsRef.set(awContents);
+        final ContentViewCore contentViewCore = testView.getContentViewCore();
+        enableJavaScriptOnUiThread(awContents);
+        awContents.getSettings().setFullscreenSupported(true);
+
+        // JS will notify this observer once it has entered fullscreen.
+        final JavascriptEventObserver jsObserver = new JavascriptEventObserver();
+        runTestOnUiThread(new Runnable() {
+            @Override
+            public void run() {
+                jsObserver.register(contentViewCore, "jsObserver");
+            }
+        });
+
+        loadUrlSync(awContents, awContentsClient.getOnPageFinishedHelper(), FULLSCREEN_TEST_URL);
+
+        assertTrue(readyToEnterFullscreenSignal.await(
+                AwTestBase.WAIT_TIMEOUT_MS, TimeUnit.MILLISECONDS));
+        DOMUtils.clickNode(VisualStateTest.this, contentViewCore, ENTER_FULLSCREEN_CONTROL_ID);
+
+        assertTrue(hasCustomViewSignal.await(AwTestBase.WAIT_TIMEOUT_MS, TimeUnit.MILLISECONDS));
+        assertTrue(jsObserver.waitForEvent(WAIT_TIMEOUT_MS));
+
+        runTestOnUiThread(new Runnable() {
+            @Override
+            public void run() {
+                awContents.insertVisualStateCallback(20, new VisualStateCallback() {
+                    @Override
+                    public void onComplete(long id) {
+                        assertFalse(customViewRef.get().isAttachedToWindow());
+                        // NOTE: We cannot use drawAwContents here because the web contents
+                        // are rendered into the custom view while in fullscreen.
+                        Bitmap redScreenshot = GraphicsTestUtils.drawView(
+                                customViewRef.get(), 100, 100);
+                        assertEquals(Color.RED, redScreenshot.getPixel(50, 50));
+                        testFinishedSignal.countDown();
+                    }
+                });
+            }
+        });
+
+        assertTrue(testFinishedSignal.await(AwTestBase.WAIT_TIMEOUT_MS, TimeUnit.MILLISECONDS));
+    }
+
     private static final LoadUrlParams createTestPageUrl(String backgroundColor) {
         return LoadUrlParams.createLoadDataParams(
                 "<html><body bgcolor=" + backgroundColor + "></body></html>", "text/html", false);
     }
 }