Modify android video_quality_loopback_test to run commands from the src dir.

webrtc/examples/androidtests/video_quality_loopback_test.py

 #!/usr/bin/env python
 # Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 """
@@ skipping 14 matching lines @@
 import subprocess
 import sys
 import tempfile
 
 
 SCRIPT_DIR = os.path.dirname(os.path.abspath(__file__))
 SRC_DIR = os.path.normpath(os.path.join(SCRIPT_DIR, os.pardir, os.pardir,
                                         os.pardir))
 
 
-def _RunCommand(argv, **kwargs):
+def _RunCommand(argv, cwd=SRC_DIR, **kwargs):

[kjellander (google.com), 2017/02/14 12:31:05]

The script supports --source_dir flag, which overrides this (I don't remember
the use case for it though, you could ask mandermo@). But either way I think we
should use the value of args.source_dir instead, or just remove the flag
entirely.

[mandermo, 2017/02/14 14:21:15]

Was useful during the development to provide the source_dir. I agree that either
args.source_dir should be used or args.source_dir should be removed.

[kjellander_webrtc, 2017/02/14 14:36:11]

I say we just drop the flag then. It shouldn't be needed anymore I think.

   logging.info('Running %r', argv)
-  subprocess.check_call(argv, **kwargs)
+  subprocess.check_call(argv, cwd=cwd, **kwargs)
 
 
 def _ParseArgs():
   parser = argparse.ArgumentParser(description='Start loopback video analysis.')
   parser.add_argument('--source_dir', default=SRC_DIR,
       help='The path to the WebRTC source directory. Default: %(default)s.')
   parser.add_argument('build_dir_android',
       help='The path to the build directory for Android.')
   parser.add_argument('--build_dir_x86',
       help='The path to the build directory for building locally.')
@@ skipping 79 matching lines @@
       '--stats_file_ref', stats_file_ref,
       '--stats_file_test', stats_file_test, '--frame_analyzer', frame_analyzer,
       '--ffmpeg_path', ffmpeg_path, '--zxing_path', zxing_path])
 
   shutil.rmtree(temp_dir)
 
 
 if __name__ == '__main__':
   sys.exit(main())