Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8890)

Unified Diff: chrome/test/data/extensions/api_test/webrequest/test_types.js

Issue 2402233002: Tag CSP violation reports as CSP reports, not as ping (Closed)
Patch Set: Rebase on patch for crbug.com/129353 Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/api_test/webrequest/test_types.js
diff --git a/chrome/test/data/extensions/api_test/webrequest/test_types.js b/chrome/test/data/extensions/api_test/webrequest/test_types.js
index 561cd0c5cce3c75545f1e56f6fcdb71e44d95d74..5bf86cb63ccedbc27bd8e9d329ed40c789c85c20 100644
--- a/chrome/test/data/extensions/api_test/webrequest/test_types.js
+++ b/chrome/test/data/extensions/api_test/webrequest/test_types.js
@@ -35,6 +35,12 @@ function getBeaconURL() {
return getServerURL('empty.html?as-beacon');
}
+function getCSPReportURL() {
+ // dont-ignore-me is included so that framework.js does not filter out the
+ // request of type "other".
+ return getServerURL('csp-violation-dont-ignore-me');
+}
+
// A slow URL used for the beacon test, to make sure that the test fails
// deterministically if there is a bug that causes the frameId/tabId to not be
// set if the frame is removed during the request.
@@ -610,4 +616,94 @@ runTests([
};
frame.remove();
},
+
+ function typeOther_cspreport() {
+ expect([
+ { label: 'onBeforeRequest',
+ event: 'onBeforeRequest',
+ details: {
+ type: 'csp_report',
+ method: 'POST',
+ url: getCSPReportURL(),
+ frameUrl: 'unknown frame URL',
+ frameId: 1,
+ parentFrameId: 0,
+ tabId: 1,
+ }
+ },
+ { label: 'onBeforeSendHeaders',
+ event: 'onBeforeSendHeaders',
+ details: {
+ type: 'csp_report',
+ method: 'POST',
+ url: getCSPReportURL(),
+ frameId: 1,
+ parentFrameId: 0,
+ tabId: 1,
+ },
+ },
+ { label: 'onSendHeaders',
+ event: 'onSendHeaders',
+ details: {
+ type: 'csp_report',
+ method: 'POST',
+ url: getCSPReportURL(),
+ frameId: 1,
+ parentFrameId: 0,
+ tabId: 1,
+ },
+ },
+ { label: 'onHeadersReceived',
+ event: 'onHeadersReceived',
+ details: {
+ type: 'csp_report',
+ method: 'POST',
+ url: getCSPReportURL(),
+ frameId: 1,
+ parentFrameId: 0,
+ tabId: 1,
+ statusLine: 'HTTP/1.1 404 Not Found',
+ statusCode: 404,
+ },
+ },
+ { label: 'onResponseStarted',
+ event: 'onResponseStarted',
+ details: {
+ type: 'csp_report',
+ method: 'POST',
+ url: getCSPReportURL(),
+ frameId: 1,
+ parentFrameId: 0,
+ tabId: 1,
+ ip: '127.0.0.1',
+ fromCache: false,
+ statusLine: 'HTTP/1.1 404 Not Found',
+ statusCode: 404,
+ },
+ },
+ { label: 'onCompleted',
+ event: 'onCompleted',
+ details: {
+ type: 'csp_report',
+ method: 'POST',
+ url: getCSPReportURL(),
+ frameId: 1,
+ parentFrameId: 0,
+ tabId: 1,
+ ip: '127.0.0.1',
+ fromCache: false,
+ statusLine: 'HTTP/1.1 404 Not Found',
+ statusCode: 404,
+ },
+ }],
+ [['onBeforeRequest', 'onBeforeSendHeaders', 'onSendHeaders',
+ 'onHeadersReceived', 'onResponseStarted', 'onCompleted']], {
+ urls: ['<all_urls>'], types: ['csp_report']
+ });
+
+ var frame = document.createElement('iframe');
+ frame.src =
+ getServerURL('extensions/api_test/webrequest/csp/violation.html');
+ document.body.appendChild(frame);
+ },
]);

Powered by Google App Engine
This is Rietveld 408576698