Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(65)

Side by Side Diff: media/mojo/scripts/run_mojo_media_renderer.py

Issue 1088993004: Update run_mojo_media_renderer.py for shell and service changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove unnecessary origin and mapping args, broken headless flag; update comments. Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2015 The Chromium Authors. All rights reserved. 2 # Copyright 2015 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 # 5 #
6 # The script follows mojo/services/html_viewer/view_url.py and modified it for 6 # The script launches mojo_shell to test the mojo media renderer.
7 # test the mojo media renderer. The page will be rendered in a headless mode.
8 #
9 # TODO(xhwang): Explore the possibility of running this with the Kiosk window
10 # manager. See http://crbug.com/467176
11 7
12 import argparse 8 import argparse
13 import os 9 import os
14 import subprocess 10 import subprocess
15 import sys 11 import sys
16 12
13
17 root_path = os.path.realpath( 14 root_path = os.path.realpath(
18 os.path.join( 15 os.path.join(
19 os.path.dirname( 16 os.path.dirname(
20 os.path.realpath(__file__)), 17 os.path.realpath(__file__)),
21 os.pardir, 18 os.pardir,
22 os.pardir, 19 os.pardir,
23 os.pardir)) 20 os.pardir))
24 21
22
25 def _BuildShellCommand(args): 23 def _BuildShellCommand(args):
26 sdk_version = subprocess.check_output(["cat", 24 sdk_version = subprocess.check_output(["cat",
27 "third_party/mojo/src/mojo/public/VERSION"], cwd=root_path) 25 "third_party/mojo/src/mojo/public/VERSION"], cwd=root_path)
28 build_dir = os.path.join(root_path, args.build_dir) 26 build_dir = os.path.join(root_path, args.build_dir)
29 27
30 shell_command = [os.path.join(build_dir, "mojo_shell")] 28 shell_command = [os.path.join(build_dir, "mojo_shell")]
31 29
32 options = [] 30 options = []
33 31
34 options.append( 32 options.append("--enable-mojo-media-renderer")
sky 2015/04/23 19:45:37 nit: I would move to this 30, eg: options = [ "--e
msw 2015/04/23 19:59:02 Done.
35 "--origin=https://storage.googleapis.com/mojo/services/linux-x64/%s" %
36 sdk_version)
37 options.append("--url-mappings=mojo:html_viewer=file://%s/html_viewer.mojo,"
38 "mojo:media=file://%s/media.mojo" % (build_dir, build_dir))
39
40 args_for_html_viewer = "--enable-mojo-media-renderer --is-headless "
41 if args.verbose: 33 if args.verbose:
42 args_for_html_viewer += \ 34 options.append(
43 "--vmodule=pipeline*=3,*renderer_impl*=3,*mojo_demuxer*=3" 35 "--vmodule=pipeline*=3,*renderer_impl*=3,*mojo_demuxer*=3")
44 options.append("--args-for=mojo:html_viewer %s" % args_for_html_viewer) 36 options.append("--vmodule=mojo*service=3")
45
46 if args.verbose:
47 args_for_media = "--vmodule=mojo*service=3"
48 options.append("--args-for=mojo:media %s" % args_for_media)
49 37
50 full_command = shell_command + options + [args.url] 38 full_command = shell_command + options + [args.url]
51 39
52 if args.verbose: 40 if args.verbose:
53 print full_command 41 print full_command
54 42
55 return full_command 43 return full_command
56 44
45
57 def main(): 46 def main():
58 parser = argparse.ArgumentParser( 47 parser = argparse.ArgumentParser(
59 description="View a URL with HTMLViewer with mojo media renderer. " 48 description="View a URL with HTMLViewer with mojo media renderer. "
60 "You must have built //mojo/services/html_viewer, " 49 "You must have built //components/html_viewer, "
61 "//mojo/services/network and //media/mojo/services first. " 50 "//mojo/services/network and //media/mojo/services first. "
62 " Note that this will currently often fail spectacularly due " 51 " Note that this will currently often fail spectacularly due "
63 " to lack of binary stability in Mojo.") 52 " to lack of binary stability in Mojo.")
64 parser.add_argument( 53 parser.add_argument(
65 "--build-dir", 54 "--build-dir",
66 help="Path to the dir containing the linux-x64 binaries relative to the " 55 help="Path to the dir containing the linux-x64 binaries relative to the "
67 "repo root (default: %(default)s)", 56 "repo root (default: %(default)s)",
68 default="out/Release") 57 default="out/Release")
69 parser.add_argument("--verbose", help="Increase output verbosity.", 58 parser.add_argument("--verbose", help="Increase output verbosity.",
70 action="store_true") 59 action="store_true")
71 parser.add_argument("url", 60 parser.add_argument("url", help="The URL to be viewed")
72 help="The URL to be viewed")
73 61
74 args = parser.parse_args() 62 args = parser.parse_args()
75 return subprocess.call(_BuildShellCommand(args)) 63 return subprocess.call(_BuildShellCommand(args))
76 64
77 if __name__ == '__main__': 65 if __name__ == '__main__':
78 sys.exit(main()) 66 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698