mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Bug 1530408 disable alt-data if possible when creating a StreamFilter r=valentin,zombie 

If an extension creates a stream filter early enough, we can
disable the use of the alt-data cache allowing the extension to properly
handle filtering.

Differential Revision: https://phabricator.services.mozilla.com/D68641

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Shane Caraveo 2020-04-11 00:19:07 +00:00
Родитель 43f7d20087
Коммит 587f136929
3 изменённых файлов: 62 добавлений и 5 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

1
toolkit/components/extensions/test/xpcshell/data/file_script.html
Просмотреть файл

@ -4,6 +4,7 @@
<head>
<meta charset="utf-8">
<script type="application/javascript" src="file_script_good.js"></script>
<script type="application/javascript" src="file_script_bad.js"></script>
</head>
<body>

62
toolkit/components/extensions/test/xpcshell/test_ext_webRequest_filterResponseData.js
Просмотреть файл

@ -352,23 +352,71 @@ add_task(async function test_alternate_cached_data() {
let extension = ExtensionTestUtils.loadExtension({
background() {
browser.webRequest.onHeadersReceived.addListener(
browser.webRequest.onBeforeRequest.addListener(
details => {
let filter = browser.webRequest.filterResponseData(details.requestId);
let decoder = new TextDecoder("utf-8");
let encoder = new TextEncoder();
filter.ondata = event => {
browser.test.fail(`ondata received when alt-data should be used`);
let str = decoder.decode(event.data, { stream: true });
filter.write(encoder.encode(str));
filter.disconnect();
browser.test.assertTrue(
str.startsWith(`"use strict";`),
"ondata received decoded data"
);
browser.test.sendMessage("onBeforeRequest");
};
filter.onerror = () => {
// onBeforeRequest will always beat the cache race, so we should always
// get valid data in ondata.
browser.test.fail("error-received", filter.error);
};
},
{
urls: ["http://example.com/data/file_script_good.js"],
},
["blocking"]
);
browser.webRequest.onHeadersReceived.addListener(
details => {
let filter = browser.webRequest.filterResponseData(details.requestId);
let decoder = new TextDecoder("utf-8");
let encoder = new TextEncoder();
// Because cache is always a race, intermittently we will succesfully
// beat the cache, in which case we pass in ondata. If cache wins,
// we pass in onerror.
// Running the test with --verify hits this cache race issue, as well
// it seems that the cache primarily looses on linux1804.
let gotone = false;
filter.ondata = event => {
browser.test.assertFalse(gotone, "cache lost the race");
gotone = true;
let str = decoder.decode(event.data, { stream: true });
filter.write(encoder.encode(str));
filter.disconnect();
browser.test.assertTrue(
str.startsWith(`"use strict";`),
"ondata received decoded data"
);
browser.test.sendMessage("onHeadersReceived");
};
filter.onerror = () => {
browser.test.assertFalse(gotone, "cache won the race");
gotone = true;
browser.test.assertEq(
filter.error,
"Channel is delivering cached alt-data"
);
browser.test.sendMessage("error-received");
browser.test.sendMessage("onHeadersReceived");
};
},
{
urls: ["http://example.com/*/file_script_good.js"],
urls: ["http://example.com/data/file_script_bad.js"],
},
["blocking"]
);
@ -384,12 +432,16 @@ add_task(async function test_alternate_cached_data() {
"http://example.com/data/file_script.html"
);
await contentPage.close();
await extension.startup();
let page_cached = await await ExtensionTestUtils.loadContentPage(
"http://example.com/data/file_script.html"
);
await extension.awaitMessage("error-received");
await Promise.all([
extension.awaitMessage("onBeforeRequest"),
extension.awaitMessage("onHeadersReceived"),
]);
await page_cached.close();
await extension.unload();

4
toolkit/components/extensions/webrequest/StreamFilterParent.cpp
Просмотреть файл

@ -124,6 +124,10 @@ auto StreamFilterParent::Create(dom::ContentParent* aContentParent,
return ChildEndpointPromise::CreateAndReject(false, __func__);
}
// Disable alt-data for extension stream listeners.
nsCOMPtr<nsIHttpChannelInternal> internal(do_QueryObject(channel));
internal->DisableAltDataCache();
return chan->AttachStreamFilter(aContentParent ? aContentParent->OtherPid()
: base::GetCurrentProcId());
}