GitLab

This is a reland of f8263a71

The earlier patch failed on the chromeos MSan bot only because that builder
uses include_js_tests = false, so was missing a transitive dep on
//chromeos/components/media_app_ui:browser_test_support .

Original change's description:
> Hook up chrome://media-app as a file handler for image/*
>
> Adds a consumer for window.launchQueue in chrome://media-app that
> opens the file and sends it to the existing loadBlob(), which sends
> the blob to the unprivileged guest app to load.
>
> Adds an integration test that ensures launch params passed via the
> AppService OpenApplication() will correctly open an image in the media
> app.
>
> SystemWebAppManagerBrowserTest is refactored away from extensions slightly.
> Rather than use extensions::browsertest_util::LaunchAppBrowser(), it uses
> LaunchService::OpenApplication(AppLaunchParams) directly. Mostly this just
> avoids having to fetch the Extension* only to use extension->id() to get the
> app_id back again.
>
> Bug: 996088, 1023742
> Change-Id: I85624e93618d82ccce1c80d04d21d81303579a99
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913282
> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
> Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> Commit-Queue: Trent Apted <tapted@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#720681}

TBR=ortuno@chromium.org

Bug: 996088, 1023742
Fixed: 1030105
Change-Id: Ifb0c66efccf0411c8bd9766e17ac61676a6a4ea3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947604
Commit-Queue: Trent Apted <tapted@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720805}