commit 6c15dab79a0536622304cc69fcb7a3b03fa29bf1
author Emilian Peev <epeev@mm-sol.com> Fri Mar 11 09:56:36 2011 +0200
committer Emilian Peev <epeev@ti.com> Mon Mar 14 09:23:29 2011 +0200
tree ca50b30bae357ea2d9091df545f2d08a4449f0f6
parent d1f971918e7fe8c9090409d41c14478158916a58

CTS: Fixes for android.hardware.cts.CameraTest#testPreviewCallback

- setPreviewDisplay() was omitted during the individual
  preview resolution tests. The camera documentation states
  that a call to either setPreviewDisplay() or setPreviewTexture()
  is necessary in order to start the preview. Apart from
  that setPreviewDisplay() is mandatory on platforms, which utilize
  an overlay for camera preview.

Change-Id: Ifd2ce8d787a8c1d6790701b9d04f13f4158227d0

tests/tests/hardware/src/android/hardware/cts/CameraTest.java

diff --git a/tests/tests/hardware/src/android/hardware/cts/CameraTest.java b/tests/tests/hardware/src/android/hardware/cts/CameraTest.java
index 45bc87e..ef8490d 100644
--- a/tests/tests/hardware/src/android/hardware/cts/CameraTest.java
+++ b/tests/tests/hardware/src/android/hardware/cts/CameraTest.java
@@ -405,10 +405,8 @@
             parameters.setPreviewSize(size.width, size.height);
             mCamera.setParameters(parameters);
             assertEquals(size, mCamera.getParameters().getPreviewSize());
-            mCamera.startPreview();
-            waitForPreviewDone();
+            checkPreviewCallback();
             assertTrue(mPreviewCallbackResult);
-            mCamera.stopPreview();
             try {
                 // Wait for a while to throw away the remaining preview frames.
                 Thread.sleep(1000);