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

Unified Diff: components/image_fetcher/image_data_fetcher_unittest.cc

Issue 2756633003: [Image Fetcher] Add support to disable cookies (Closed)
Patch Set: Created 3 years, 9 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: components/image_fetcher/image_data_fetcher_unittest.cc
diff --git a/components/image_fetcher/image_data_fetcher_unittest.cc b/components/image_fetcher/image_data_fetcher_unittest.cc
index 15039781fb4a9f2fd8f002491c44e16e8e28c2a8..9f65fd86ce80e714f291391dbe067ff6cb6c7546 100644
--- a/components/image_fetcher/image_data_fetcher_unittest.cc
+++ b/components/image_fetcher/image_data_fetcher_unittest.cc
@@ -10,6 +10,7 @@
#include "base/macros.h"
#include "base/memory/ref_counted.h"
#include "base/message_loop/message_loop.h"
+#include "net/base/load_flags.h"
#include "net/http/http_response_headers.h"
#include "net/http/http_status_code.h"
#include "net/url_request/test_url_fetcher_factory.h"
@@ -23,6 +24,8 @@ namespace {
const char kImageURL[] = "http://www.example.com/image";
const char kURLResponseData[] = "EncodedImageData";
+using testing::_;
+
} // namespace
namespace image_fetcher {
@@ -72,6 +75,10 @@ TEST_F(ImageDataFetcherTest, FetchImageData) {
// Get and configure the TestURLFetcher.
net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
ASSERT_NE(nullptr, test_url_fetcher);
+ EXPECT_FALSE(test_url_fetcher->GetLoadFlags() &
+ net::LOAD_DO_NOT_SEND_COOKIES);
+ EXPECT_FALSE(test_url_fetcher->GetLoadFlags() &
+ net::LOAD_DO_NOT_SAVE_COOKIES);
test_url_fetcher->set_status(
net::URLRequestStatus(net::URLRequestStatus::SUCCESS, net::OK));
test_url_fetcher->SetResponseString(kURLResponseData);
@@ -164,4 +171,18 @@ TEST_F(ImageDataFetcherTest, FetchImageData_MultipleRequests) {
test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
}
+TEST_F(ImageDataFetcherTest, FetchImageData_DisableCookies) {
+ image_data_fetcher_.DisableCookies();
+ image_data_fetcher_.FetchImageData(
+ GURL(kImageURL), base::Bind(&ImageDataFetcherTest::OnImageDataFetched,
+ base::Unretained(this)));
+
+ net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
+ ASSERT_NE(nullptr, test_url_fetcher);
+ EXPECT_TRUE(test_url_fetcher->GetLoadFlags() & net::LOAD_DO_NOT_SEND_COOKIES);
+ EXPECT_TRUE(test_url_fetcher->GetLoadFlags() & net::LOAD_DO_NOT_SAVE_COOKIES);
+}
+
+void DisableCookies();
+
} // namespace image_fetcher

Powered by Google App Engine
This is Rietveld 408576698