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

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

Issue 2694833003: Revert of Tag CSP violation reports as CSP reports, not as ping (Closed)
Patch Set: 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 68040ba14680d63180318ab95a777332939d3d21..bef3309d95a1635e587e778f4b275672961c3d74 100644
--- a/chrome/test/data/extensions/api_test/webrequest/test_types.js
+++ b/chrome/test/data/extensions/api_test/webrequest/test_types.js
@@ -30,12 +30,6 @@
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
@@ -619,94 +613,4 @@
};
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