commit dfd29495f8e67840de5acd28653e353c730ebc78
author Hirokazu Honda <hiroh@chromium.org> Fri Aug 31 23:53:38 2018 +0000
committer chrome-bot <chrome-bot@chromium.org> Sun Sep 02 07:19:04 2018 -0700
tree 59b82eb1994fb37275cc29fd03b0803662972705
parent a6b112ea7e900ab66d328109f90617dc44424f08

Revert "video_VideoDecodeAccelerator: test IMPORT mode"

TBR=deanliao@google.com

This reverts commit 9056f4eec66e651eca14dc7d4c461edcdeea37c3.

Reason for revert: This CL cause test failures at some devices.

Original change's description:
> video_VideoDecodeAccelerator: test IMPORT mode
>
> video_VideoDecodeAccelerator runs Chrome VDA unittest.
> VDA unittest tests ALLOCATE mode by default and has the option to test IMPORT
> mode, --test_import.
> video_VideoDecodeAccelerator should test not only ALLOCATE mode but also IMPORT
> mode.
>
> BUG=chromium:856562
> TEST=video_VideoDecodeAccelerator at peach_pit
>
> Change-Id: I1e58382ac41feb4d6dc0da26b74e0e3874e141ed
> Reviewed-on: https://chromium-review.googlesource.com/1186391
> Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
> Tested-by: Hirokazu Honda <hiroh@chromium.org>
> Reviewed-by: Shuo-Peng Liao <deanliao@google.com>

Bug: chromium:856562
Change-Id: I0680e48494bb8e5b4ae5781a5beefd44836d99e7
Reviewed-on: https://chromium-review.googlesource.com/1200542
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py

diff --git a/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py b/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py
index 3af76d7..6128a7c 100644
--- a/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py
+++ b/client/site_tests/video_VideoDecodeAccelerator/video_VideoDecodeAccelerator.py
@@ -55,23 +55,14 @@
             if gtest_filter:
                 cmd_line_list.append('--gtest_filter="%s"' % gtest_filter)
 
+            cmd_line = ' '.join(cmd_line_list)
             try:
-                cmd_line = ' '.join(cmd_line_list)
-                self.run_chrome_test_binary(self.binary, cmd_line)
-
-                # Test import mode
-                cmd_line_list.append('--test_import')
-                cmd_line = ' '.join(cmd_line_list)
                 self.run_chrome_test_binary(self.binary, cmd_line)
             except error.TestFail as test_failure:
                 # Continue to run the remaining test videos and raise
                 # the last failure after finishing all videos.
-                mode = '[ALLOCATE]'
-                if '--test_import' in cmd_line_list:
-                    mode = '[IMPORT]'
-                logging.error('%s %s: %s', mode, video, test_failure.message)
-                last_test_failure = error.TestFail(mode + ' ' +
-                                                   test_failure.message)
+                logging.error('%s: %s', video, test_failure.message)
+                last_test_failure = test_failure
 
         if last_test_failure:
             raise last_test_failure