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
87 lines
2.7 KiB
HTML
87 lines
2.7 KiB
HTML
<!DOCTYPE html>
|
|
<title>Sub Apps: Error cases for remove()</title>
|
|
<script src="/resources/testharness.js"></script>
|
|
<script src="/resources/testharnessreport.js"></script>
|
|
<script src="resources/subapps-helpers.js"></script>
|
|
|
|
<body></body>
|
|
|
|
<script>
|
|
|
|
promise_test(async t => {
|
|
const iframe = document.createElement('iframe');
|
|
document.body.appendChild(iframe);
|
|
|
|
const iframeNavigator = iframe.contentWindow.navigator;
|
|
const iframeDOMException = iframe.contentWindow.DOMException;
|
|
|
|
// Detach the frame.
|
|
iframe.remove();
|
|
|
|
// At this point the iframe is detached and unloaded, and its execution
|
|
// context is gone.
|
|
await promise_rejects_dom(t, 'NotFoundError', iframeDOMException, iframeNavigator.subApps.remove(['/sub-app-id']));
|
|
}, "The object is no longer associated to a document.");
|
|
|
|
promise_test(async t => {
|
|
const iframe = document.createElement('iframe');
|
|
document.body.appendChild(iframe);
|
|
|
|
const iframeNavigator = iframe.contentWindow.navigator;
|
|
const iframeDOMException = iframe.contentWindow.DOMException;
|
|
t.add_cleanup(() => iframe.remove());
|
|
|
|
await promise_rejects_dom(t, 'InvalidStateError', iframeDOMException, iframeNavigator.subApps.remove(['/sub-app-id']));
|
|
}, "API is only supported in top-level browsing contexts.");
|
|
|
|
promise_test(async t => {
|
|
const full_url = document.location.origin + '/sub-app-id';
|
|
|
|
await promise_rejects_dom(t, 'NotSupportedError', navigator.subApps.remove([full_url]));
|
|
}, 'API supports only root-relative paths.');
|
|
|
|
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.FAILURE },
|
|
{ "manifestIdPath": url_2, "resultCode": Status.FAILURE },
|
|
{ "manifestIdPath": url_3, "resultCode": Status.FAILURE }
|
|
];
|
|
|
|
let expected_results = {
|
|
[url_1]: "failure",
|
|
[url_2]: "failure",
|
|
[url_3]: "failure"
|
|
};
|
|
|
|
await subapps_remove_expect_reject_with_result(t, remove_call_params, mocked_response, expected_results);
|
|
}, 'Remove call fails.');
|
|
|
|
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.FAILURE }
|
|
];
|
|
|
|
let expected_results = {
|
|
[url_1]: "success",
|
|
[url_2]: "success",
|
|
[url_3]: "failure"
|
|
};
|
|
|
|
await subapps_remove_expect_reject_with_result(t, remove_call_params, mocked_response, expected_results);
|
|
}, 'Remove call fails with mixed results.');
|
|
|
|
</script> |