diff options
author | Igor Murashkin <iam@google.com> | 2013-03-01 02:20:44 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2013-03-01 02:20:45 +0000 |
commit | 48c3f1f982f54ebd034cfe6a1b9356d9193afb69 (patch) | |
tree | b8d95d195b86ad1df4c57194ee7eb4ca9e6f4183 | |
parent | 7072e03ae4eb9b8f48a567dbb2a5cc8d2049f331 (diff) | |
parent | 721f17283e2493426c47bbaa267c337b2af5726e (diff) | |
download | frameworks_av-48c3f1f982f54ebd034cfe6a1b9356d9193afb69.zip frameworks_av-48c3f1f982f54ebd034cfe6a1b9356d9193afb69.tar.gz frameworks_av-48c3f1f982f54ebd034cfe6a1b9356d9193afb69.tar.bz2 |
Merge "ProCameraTest: Drop frames test move processing delay between lock/unlock" into jb-mr2-dev
-rw-r--r-- | camera/tests/ProCameraTests.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/camera/tests/ProCameraTests.cpp b/camera/tests/ProCameraTests.cpp index f93e5cd..39456af 100644 --- a/camera/tests/ProCameraTests.cpp +++ b/camera/tests/ProCameraTests.cpp @@ -1001,11 +1001,6 @@ TEST_F(ProCameraTest, WaitForSingleStreamBufferAndDropFrames) { // Consume a couple of results for (int i = 0; i < NUM_REQUESTS; ++i) { - // Process at 10fps, stream is at 15fps. - // This means we will definitely fill up the buffer queue with - // extra buffers and need to drop them. - usleep(TEST_FRAME_PROCESSING_DELAY_US); - int numFrames; EXPECT_TRUE((numFrames = mCamera->waitForFrameBuffer(streamId)) > 0); @@ -1025,6 +1020,11 @@ TEST_F(ProCameraTest, WaitForSingleStreamBufferAndDropFrames) { ", dataPtr = " << (void*)buf.data << ", timestamp = " << buf.timestamp << std::endl; + // Process at 10fps, stream is at 15fps. + // This means we will definitely fill up the buffer queue with + // extra buffers and need to drop them. + usleep(TEST_FRAME_PROCESSING_DELAY_US); + EXPECT_OK(consumer->unlockBuffer(buf)); } |