DevTools: fix Target.disposeBrowserContext() to work with empty contexts
Currently, chrome's implementation of Target.disposeBrowserContext()
fails to dispose empty browser contexts.
We don't need to wait for browsers to close if there are no opened
browser windows to destroy a browser context.
R=dgozman
Change-Id: I7b41a1b55b398f543a547fd6ead9f83af7f8a843
Reviewed-on: https://chromium-review.googlesource.com/1050909
Reviewed-by: Andrey Kosyakov <[email protected]>
Commit-Queue: Andrey Lushnikov <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#557085}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 0c5d7cecdef91c4185475813ef01a41137422449diff --git a/front_end/Tests.js b/front_end/Tests.js
index 3fbff04..d853042 100644
--- a/front_end/Tests.js
+++ b/front_end/Tests.js
@@ -1110,6 +1110,18 @@
this.releaseControl();
};
+ TestSuite.prototype.testDisposeEmptyBrowserContext = async function(url) {
+ this.takeControl();
+ const targetAgent = SDK.targetManager.mainTarget().targetAgent();
+ const {browserContextId} = await targetAgent.invoke_createBrowserContext();
+ const response1 = await targetAgent.invoke_getBrowserContexts();
+ this.assertEquals(response1.browserContextIds.length, 1);
+ await targetAgent.invoke_disposeBrowserContext({browserContextId});
+ const response2 = await targetAgent.invoke_getBrowserContexts();
+ this.assertEquals(response2.browserContextIds.length, 0);
+ this.releaseControl();
+ };
+
TestSuite.prototype.testCreateBrowserContext = async function(url) {
this.takeControl();
const browserContextIds = [];
