mirror of
https://github.com/HeyPuter/firefox.git
synced 2026-11-13 20:14:57 -10:00
Daniel Murphy
f6f39c280e
Automatic update from web-platform-tests [dPWA] Refactoring relative manifest id to absolute id. This patch refactors the manifest parser & id handling to use the computed manifest_id defined in the spec, which simplifies a lot of code. This patch: - Makes changes to the ManifestParser to output the id as a full url. - Always populates the manifest_id web app representations. - Fixes a variety of tests, including some small mistakes in the sub app tests that are hopefully minor. - Adds a number of checks to ensure that things are created correctly. - Updates the web_app_helper methods appropriately. This change should have no functional impact. Bug: 1231765,1442799,b/280606515 Change-Id: I6d69e2d94af9278d796df4efb3e275416e4ad59e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4515997 Reviewed-by: Dibyajyoti Pal <[email protected]> Reviewed-by: Maksim Moskvitin <[email protected]> Reviewed-by: Marc Treib <[email protected]> Reviewed-by: Tom Sepez <[email protected]> Commit-Queue: Daniel Murphy <[email protected]> Cr-Commit-Position: refs/heads/main@{#1144248} -- wpt-commits: cb1e54222a206b4cbb5211f7e24878a267b820f1 wpt-pr: 39948
41 lines
1.2 KiB
HTML
41 lines
1.2 KiB
HTML
<!DOCTYPE html>
|
|
<title>Sub Apps: Valid calls for remove()</title>
|
|
<script src="/resources/testharness.js"></script>
|
|
<script src="/resources/testharnessreport.js"></script>
|
|
<script src="resources/subapps-helpers.js"></script>
|
|
<script>
|
|
|
|
promise_test(async t => {
|
|
|
|
const url = '/sub-app';
|
|
let remove_call_params = [url];
|
|
let mocked_response = [{ "manifestIdPath": url, "resultCode": Status.SUCCESS }];
|
|
let expected_results = {[url]: "success"};
|
|
|
|
await subapps_remove_expect_success_with_result(t, remove_call_params, mocked_response, expected_results);
|
|
}, 'Remove API call works with one app.');
|
|
|
|
promise_test(async t => {
|
|
|
|
const url_1 = '/sub-app-1';
|
|
const url_2 = '/sub-app-2';
|
|
const url_3 = '/sub-app-3';
|
|
|
|
let remove_call_params = [url_1, url_2, url_3];
|
|
|
|
let mocked_response = [
|
|
{ "manifestIdPath": url_1, "resultCode": Status.SUCCESS },
|
|
{ "manifestIdPath": url_2, "resultCode": Status.SUCCESS },
|
|
{ "manifestIdPath": url_3, "resultCode": Status.SUCCESS }
|
|
];
|
|
|
|
let expected_results = {
|
|
[url_1]: "success",
|
|
[url_2]: "success",
|
|
[url_3]: "success"
|
|
};
|
|
|
|
await subapps_remove_expect_success_with_result(t, remove_call_params, mocked_response, expected_results);
|
|
}, 'Remove API call works with several apps.');
|
|
|
|
</script> |