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

Unified Diff: content/browser/background_fetch/background_fetch_service_unittest.cc

Issue 2805813006: Set the Content-Type and Content-Length headers for BG Fetch (Closed)
Patch Set: Set the Content-Type and Content-Length headers for BG Fetch Created 3 years, 8 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: content/browser/background_fetch/background_fetch_service_unittest.cc
diff --git a/content/browser/background_fetch/background_fetch_service_unittest.cc b/content/browser/background_fetch/background_fetch_service_unittest.cc
index 8e47edd3c89f1582d2b511744ecbe50fdaafff86..a76ac7a1f340b52475f5fdc524dd89eed7031de1 100644
--- a/content/browser/background_fetch/background_fetch_service_unittest.cc
+++ b/content/browser/background_fetch/background_fetch_service_unittest.cc
@@ -369,11 +369,11 @@ TEST_F(BackgroundFetchServiceTest, FetchSuccessEventDispatch) {
EXPECT_EQ(fetches[i].request.url, fetches[i].response.url_list[0]);
// TODO(peter): change-detector tests for unsupported properties.
- EXPECT_EQ(fetches[i].response.status_code, 0);
+ EXPECT_EQ(fetches[i].response.status_code, 200);
EXPECT_TRUE(fetches[i].response.status_text.empty());
EXPECT_EQ(fetches[i].response.response_type,
blink::WebServiceWorkerResponseTypeDefault);
- EXPECT_TRUE(fetches[i].response.headers.empty());
+ EXPECT_FALSE(fetches[i].response.headers.empty());
EXPECT_EQ(fetches[i].response.error,
blink::WebServiceWorkerResponseErrorUnknown);

Powered by Google App Engine
This is Rietveld 408576698