Commit daf3ffc1 authored by Jiewei Qian's avatar Jiewei Qian Committed by Chromium LUCI CQ

autotestPrivate: add publisher_id to getAllInstalledApps

This CL exposes AppService App publisher_id to Tast API. publisher_id
can be used to identify an App in the app publisher's subsystem. For
example, Andrdoid package name for ARC, Web app start url for Web Apps.

This is useful for tast tests wishing to interact with the App outside
of the App Service infrastructure.

Fixed: 1155337
Change-Id: I6719526a1141b312d1f8758d61b2ce716889c68f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580965Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Reviewed-by: default avatarToni Baržić <tbarzic@chromium.org>
Commit-Queue: Jiewei Qian  <qjw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835987}
parent 5e757964
...@@ -3057,6 +3057,7 @@ AutotestPrivateGetAllInstalledAppsFunction::Run() { ...@@ -3057,6 +3057,7 @@ AutotestPrivateGetAllInstalledAppsFunction::Run() {
app.app_id = update.AppId(); app.app_id = update.AppId();
app.name = update.Name(); app.name = update.Name();
app.short_name = update.ShortName(); app.short_name = update.ShortName();
app.publisher_id = update.PublisherId();
app.additional_search_terms = update.AdditionalSearchTerms(); app.additional_search_terms = update.AdditionalSearchTerms();
app.type = GetAppType(update.AppType()); app.type = GetAppType(update.AppType());
app.install_source = GetAppInstallSource(update.InstallSource()); app.install_source = GetAppInstallSource(update.InstallSource());
......
...@@ -323,6 +323,7 @@ namespace autotestPrivate { ...@@ -323,6 +323,7 @@ namespace autotestPrivate {
DOMString appId; DOMString appId;
DOMString name; DOMString name;
DOMString shortName; DOMString shortName;
DOMString publisherId;
AppType? type; AppType? type;
AppInstallSource? installSource; AppInstallSource? installSource;
AppReadiness? readiness; AppReadiness? readiness;
......
...@@ -500,6 +500,7 @@ var defaultTests = [ ...@@ -500,6 +500,7 @@ var defaultTests = [
// their values change if chrome_branded is true. // their values change if chrome_branded is true.
chrome.test.assertTrue(!!chromium.name); chrome.test.assertTrue(!!chromium.name);
chrome.test.assertTrue(!!chromium.shortName); chrome.test.assertTrue(!!chromium.shortName);
chrome.test.assertEq(chromium.publisherId, "");
chrome.test.assertEq(chromium.additionalSearchTerms, []); chrome.test.assertEq(chromium.additionalSearchTerms, []);
chrome.test.assertEq(chromium.readiness, 'Ready'); chrome.test.assertEq(chromium.readiness, 'Ready');
chrome.test.assertEq(chromium.showInLauncher, true); chrome.test.assertEq(chromium.showInLauncher, true);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment