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

Unified Diff: webkit/glue/multipart_response_delegate_unittest.cc

Issue 10440014: Correctly whitelist ftp directory listings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 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: webkit/glue/multipart_response_delegate_unittest.cc
diff --git a/webkit/glue/multipart_response_delegate_unittest.cc b/webkit/glue/multipart_response_delegate_unittest.cc
index 474daa07bc587bb59f4af7c691be84ab89e5d8fb..a82d6fc5dbc11b519696a2d4fc0f59455dc7477a 100644
--- a/webkit/glue/multipart_response_delegate_unittest.cc
+++ b/webkit/glue/multipart_response_delegate_unittest.cc
@@ -642,6 +642,7 @@ TEST(MultipartResponseTest, MultipartPayloadSet) {
WebURLResponse response;
response.initialize();
response.setMIMEType("multipart/x-mixed-replace");
+ response.setExtraData(new WebURLResponseExtraDataImpl(""));
MockWebURLLoaderClient client;
MultipartResponseDelegate delegate(&client, NULL, response, "bound");
@@ -656,7 +657,9 @@ TEST(MultipartResponseTest, MultipartPayloadSet) {
EXPECT_EQ(1, client.received_response_);
EXPECT_EQ(string("response data"), client.data_);
EXPECT_EQ(static_cast<int>(data.length()), client.total_encoded_data_length_);
- EXPECT_FALSE(client.response_.isMultipartPayload());
+ WebURLResponseExtraDataImpl* extra_data =
+ static_cast<WebURLResponseExtraDataImpl*>(client.response_.extraData());
+ EXPECT_FALSE(extra_data->is_multipart_payload());
string data2(
"Content-type: text/plain\n\n"
@@ -669,7 +672,9 @@ TEST(MultipartResponseTest, MultipartPayloadSet) {
EXPECT_EQ(string("response data2"), client.data_);
EXPECT_EQ(static_cast<int>(data.length()) + static_cast<int>(data2.length()),
client.total_encoded_data_length_);
- EXPECT_TRUE(client.response_.isMultipartPayload());
+ extra_data = static_cast<WebURLResponseExtraDataImpl*>(
+ client.response_.extraData());
+ EXPECT_TRUE(extra_data->is_multipart_payload());
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698