Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

fix(webkit): switch to unique loader ids (#454)

Browse source
This commit is contained in:
Yury Semikhatsky 2020-01-10 15:16:06 -08:00 committed by Dmitry Gozman
parent 3216e4f024
commit fd5c872d7f
3 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
package.json
View file

@ -10,7 +10,7 @@
"playwright": {
"chromium_revision": "724623",
"firefox_revision": "1009",
"webkit_revision": "1080"
"webkit_revision": "1082"
},
"scripts": {
"unit": "node test/test.js",

2
src/webkit/wkNetworkManager.ts
View file

@ -95,7 +95,7 @@ export class WKNetworkManager {
const frame = this._page._frameManager.frame(event.frameId);
// TODO(einbinder) this will fail if we are an XHR document request
const isNavigationRequest = event.type === 'Document';
const documentId = isNavigationRequest ? this._session.sessionId + '::' + event.loaderId : undefined;
const documentId = isNavigationRequest ? event.loaderId : undefined;
const request = new InterceptableRequest(this._session, this._page._state.interceptNetwork, frame, event, redirectChain, documentId);
this._requestIdToRequest.set(event.requestId, request);
this._page._frameManager.requestStarted(request.request);

4
src/webkit/wkPage.ts
View file

@ -195,9 +195,7 @@ export class WKPage implements PageDelegate {
frame._contextDestroyed(context);
}
}
// Append session id to avoid cross-process loaderId clash.
const documentId = this._session.sessionId + '::' + framePayload.loaderId;
this._page._frameManager.frameCommittedNewDocumentNavigation(framePayload.id, framePayload.url, framePayload.name || '', documentId, initial);
this._page._frameManager.frameCommittedNewDocumentNavigation(framePayload.id, framePayload.url, framePayload.name || '', framePayload.loaderId, initial);
}
_onFrameNavigatedWithinDocument(frameId: string, url: string) {