Commit 6320a166 authored by Sam Burnett's avatar Sam Burnett Committed by Commit Bot

Test that NEL ignores policy header fields when they're the wrong type.

Bug: 901987
Change-Id: Ic0f758b08afad0462deff8c7283ce1ab1a706dae
Reviewed-on: https://chromium-review.googlesource.com/c/1320595Reviewed-by: default avatarMisha Efimov <mef@chromium.org>
Reviewed-by: default avatarDouglas Creager <dcreager@chromium.org>
Commit-Queue: Sam Burnett <sburnett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606146}
parent f009bbf9
......@@ -795,6 +795,17 @@ TEST_F(NetworkErrorLoggingServiceTest, StatusAsValue) {
service()->OnHeader(kOrigin_, kServerIP_, kHeaderSuccessFraction1);
service()->OnHeader(kOriginDifferentHost_, kServerIP_, kHeader_);
service()->OnHeader(kOriginSubdomain_, kServerIP_, kHeaderIncludeSubdomains_);
const std::string kHeaderWrongTypes =
("{\"report_to\":\"group\","
"\"max_age\":86400,"
// We'll ignore each of these fields because they're the wrong type.
// We'll use a default value instead.
"\"include_subdomains\":\"true\","
"\"success_fraction\": \"1.0\","
"\"failure_fraction\": \"0.0\"}");
service()->OnHeader(
url::Origin::Create(GURL("https://invalid-types.example.com")),
kServerIP_, kHeaderWrongTypes);
base::Value actual = service()->StatusAsValue();
std::unique_ptr<base::Value> expected = base::test::ParseJson(R"json(
......@@ -816,6 +827,14 @@ TEST_F(NetworkErrorLoggingServiceTest, StatusAsValue) {
"successFraction": 0.0,
"failureFraction": 1.0,
},
{
"origin": "https://invalid-types.example.com",
"includeSubdomains": false,
"expires": "86400000",
"reportTo": "group",
"successFraction": 0.0,
"failureFraction": 1.0,
},
{
"origin": "https://subdomain.example.com",
"includeSubdomains": true,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment