Add an Origin Trial for Navigator.getInstalledRelatedApps.

Add an Origin Trial for Navigator.getInstalledRelatedApps.

BUG=587623
Review-Url: https://codereview.chromium.org/2785913003
Cr-Commit-Position: refs/heads/master@{#461360}
Committed: https://chromium.googlesource.com/chromium/src/+/1970247d68492c3fa208468e34aab2d7803c192b

[Matt Giuca, 2017-03-31 04:50:17]

The CQ bit was checked by mgiuca@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-31 04:50:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Matt Giuca, 2017-03-31 05:33:48]

mgiuca@chromium.org changed reviewers:
+ esprehn@chromium.org

[Matt Giuca, 2017-03-31 05:33:49]

Hi Elliott. Please have a look at this change to enable origin trial.

Intent to experiment:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/j-e8iI-WWg8

Note that I didn't use origin_trial_os = android (it's available on all
platforms). On non-Android platforms it just returns the empty list, and I think
it's good to trial that behaviour along with the working Android implementation
(as opposed to conducting a trial where it's missing on non-Android platforms
then rolling out a different behaviour later).

[esprehn, 2017-03-31 06:00:55]

Lgtm though the runtime flag and origin trial names are confusing with stuff
like web apks. I wonder if it should have been ListInstalledRelatedApps or
something.

[commit-bot: I haz the power, 2017-03-31 06:44:20]

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

[commit-bot: I haz the power, 2017-03-31 06:44:21]

Dry run: This issue passed the CQ dry run.

[Matt Giuca, 2017-04-03 00:21:21]

On 2017/03/31 06:00:55, esprehn wrote:
> Lgtm though the runtime flag and origin trial names are confusing with stuff
> like web apks. I wonder if it should have been ListInstalledRelatedApps or
> something.

The entire feature is called "installedapp" (the directory names, etc) and has
been named this since before I started work on it. I agree that isn't really
optimal especially since there's a sibling directory called "installation", but
we would have to rename a lot of stuff at this point and I don't think it's
worth it.

[Matt Giuca, 2017-04-03 00:45:35]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2017-04-03 00:45:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-03 02:10:49]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491180335198550,
"parent_rev": "fb473a87fb9a0cb77a70811505929ba08c580e74", "commit_rev":
"1970247d68492c3fa208468e34aab2d7803c192b"}

[commit-bot: I haz the power, 2017-04-03 02:11:56]

Description was changed from

==========
Add an Origin Trial for Navigator.getInstalledRelatedApps.

BUG=587623
==========

to

==========
Add an Origin Trial for Navigator.getInstalledRelatedApps.

BUG=587623

Review-Url: https://codereview.chromium.org/2785913003
Cr-Commit-Position: refs/heads/master@{#461360}
Committed:
https://chromium.googlesource.com/chromium/src/+/1970247d68492c3fa208468e34aa...
==========

[commit-bot: I haz the power, 2017-04-03 02:11:57]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1970247d68492c3fa208468e34aa...