Update run_mojo_media_renderer.py for shell and service changes.

Update run_mojo_media_renderer.py for shell and service changes.

Updates the script to handle recent changes:
  html_viewer has moved to components.
  kiosk_wm is run by default.
  services built locally, don't need origin mapping?
  avoid html_document.cc:150 Check failed: !is_headless_
  --args-for is deprecated (pass app args as shell args)

Also update mojo_media_application.cc arg init.

Tested that this works with:
  gn gen out/Debug --args='is_debug=true use_goma=true proprietary_codecs=true ffmpeg_branding="ChromeOS" enable_media_mojo_renderer=true'
  ninja -C out/Debug/ -j 1500 -l 10 mojo media/mojo/services components/html_viewer mojo/services/network
  media/mojo/scripts/run_mojo_media_renderer.py --build-dir out/Debug --verbose http://wxh.kir/ensemble/kqed.html

BUG=NONE
TEST=Script works (or closer to working...)
R=sky@chromium.org,xhwang@chromium.org

Committed: https://crrev.com/5ea51a1f1ce3b2e200fc95b232120c1582f31eab
Cr-Commit-Position: refs/heads/master@{#326690}

[msw, 2015-04-23 19:39:24]

msw@chromium.org changed reviewers:
+ xhwang@chromium.org

[msw, 2015-04-23 19:42:25]

Please take a look; thanks!

[sky, 2015-04-23 19:45:37]

LGTM

https://codereview.chromium.org/1088993004/diff/40001/media/mojo/scripts/run_...
File media/mojo/scripts/run_mojo_media_renderer.py (right):

https://codereview.chromium.org/1088993004/diff/40001/media/mojo/scripts/run_...
media/mojo/scripts/run_mojo_media_renderer.py:32:
options.append("--enable-mojo-media-renderer")
nit: I would move to this 30, eg:
options = [ "--ena..." ]

[msw, 2015-04-23 19:59:02]

https://codereview.chromium.org/1088993004/diff/40001/media/mojo/scripts/run_...
File media/mojo/scripts/run_mojo_media_renderer.py (right):

https://codereview.chromium.org/1088993004/diff/40001/media/mojo/scripts/run_...
media/mojo/scripts/run_mojo_media_renderer.py:32:
options.append("--enable-mojo-media-renderer")
On 2015/04/23 19:45:37, sky wrote:
> nit: I would move to this 30, eg:
> options = [ "--ena..." ]

Done.

[msw, 2015-04-23 19:59:17]

The CQ bit was checked by msw@chromium.org

[msw, 2015-04-23 19:59:17]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1088993004/#ps60001
(title: "Inline --enable-mojo-media-renderer option.")

[commit-bot: I haz the power, 2015-04-23 19:59:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1088993004/60001

[commit-bot: I haz the power, 2015-04-23 20:09:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-23 20:09:04]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[msw, 2015-04-23 20:10:46]

Xiaohan, please take a look; thanks!

[xhwang, 2015-04-23 22:18:49]

Verified and it's working. Thanks!

LGTM % one tiny nit

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
File media/mojo/scripts/run_mojo_media_renderer.py (right):

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
media/mojo/scripts/run_mojo_media_renderer.py:47: "You must have built
//components/html_viewer, "
Please help add //mojo here for mojo_shell.

[xhwang, 2015-04-23 22:23:57]

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
File media/mojo/scripts/run_mojo_media_renderer.py (right):

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
media/mojo/scripts/run_mojo_media_renderer.py:34:
options.append("--vmodule=mojo*service=3")
How do I know which option is for which app?

[msw, 2015-04-23 22:54:05]

Done; please take another look. +CC John for args FYI.

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
File media/mojo/scripts/run_mojo_media_renderer.py (right):

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
media/mojo/scripts/run_mojo_media_renderer.py:34:
options.append("--vmodule=mojo*service=3")
On 2015/04/23 22:23:57, xhwang wrote:
> How do I know which option is for which app?

As discussed in IM, --args-for and app-specific arguments in general are
deprecated; all shell arguments are available to all applications via
base::CommandLine. See <https://codereview.chromium.org/1057603003/>. I merged
the --vmodule args and removed the redundant and wrong media CommandLine init
from ApplicationImpl::args as we discussed.

https://codereview.chromium.org/1088993004/diff/60001/media/mojo/scripts/run_...
media/mojo/scripts/run_mojo_media_renderer.py:47: "You must have built
//components/html_viewer, "
On 2015/04/23 22:18:49, xhwang wrote:
> Please help add //mojo here for mojo_shell.

Done.

[msw, 2015-04-23 22:56:51]

The CQ bit was checked by msw@chromium.org

[msw, 2015-04-23 22:56:51]

The patchset sent to the CQ was uploaded after l-g-t-m from xhwang@chromium.org,
sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1088993004/#ps100001
(title: "Remove out of date base::CommandLine init.")

[commit-bot: I haz the power, 2015-04-23 22:57:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1088993004/100001

[xhwang, 2015-04-23 22:58:39]

LGTM, thanks for fixing this!

[commit-bot: I haz the power, 2015-04-23 23:31:16]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-04-23 23:32:50]

Patchset 6 (id:??) landed as
https://crrev.com/5ea51a1f1ce3b2e200fc95b232120c1582f31eab
Cr-Commit-Position: refs/heads/master@{#326690}