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

Unified Diff: components/image_fetcher/image_data_fetcher.h

Issue 2761303002: Move common ImageFetcher component files to core/ (Closed)
Patch Set: Rebased. 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.h
diff --git a/components/image_fetcher/image_data_fetcher.h b/components/image_fetcher/image_data_fetcher.h
deleted file mode 100644
index fbf544f688476496cb26a4b4dd48325d84d7d605..0000000000000000000000000000000000000000
--- a/components/image_fetcher/image_data_fetcher.h
+++ /dev/null
@@ -1,84 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef COMPONENTS_IMAGE_FETCHER_IMAGE_DATA_FETCHER_H_
-#define COMPONENTS_IMAGE_FETCHER_IMAGE_DATA_FETCHER_H_
-
-#include <map>
-#include <memory>
-#include <string>
-
-#include "base/callback.h"
-#include "base/macros.h"
-#include "base/memory/ref_counted.h"
-#include "components/data_use_measurement/core/data_use_user_data.h"
-#include "components/image_fetcher/request_metadata.h"
-#include "net/url_request/url_fetcher_delegate.h"
-#include "net/url_request/url_request.h"
-#include "url/gurl.h"
-
-namespace net {
-class URLFetcher;
-class URLRequestContextGetter;
-} // namespace net
-
-namespace image_fetcher {
-
-class ImageDataFetcher : public net::URLFetcherDelegate {
- public:
-
- // Callback with the |image_data|. If an error prevented a http response,
- // |request_metadata.response_code| will be RESPONSE_CODE_INVALID.
- using ImageDataFetcherCallback =
- base::Callback<void(const std::string& image_data,
- const RequestMetadata& request_metadata)>;
-
- using DataUseServiceName = data_use_measurement::DataUseUserData::ServiceName;
-
- explicit ImageDataFetcher(
- net::URLRequestContextGetter* url_request_context_getter);
- ~ImageDataFetcher() override;
-
- // Sets a service name against which to track data usage.
- void SetDataUseServiceName(DataUseServiceName data_use_service_name);
-
- // Fetches the raw image bytes from the given |image_url| and calls the given
- // |callback|. The callback is run even if fetching the URL fails. In case
- // of an error an empty string is passed to the callback.
- void FetchImageData(const GURL& image_url,
- const ImageDataFetcherCallback& callback);
-
- // Like above, but lets the caller set a referrer.
- void FetchImageData(const GURL& image_url,
- const ImageDataFetcherCallback& callback,
- const std::string& referrer,
- net::URLRequest::ReferrerPolicy referrer_policy);
-
- private:
- struct ImageDataFetcherRequest;
-
- // Method inherited from URLFetcherDelegate
- void OnURLFetchComplete(const net::URLFetcher* source) override;
-
- // All active image url requests.
- std::map<const net::URLFetcher*, std::unique_ptr<ImageDataFetcherRequest>>
- pending_requests_;
-
- scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
-
- DataUseServiceName data_use_service_name_;
-
- // The next ID to use for a newly created URLFetcher. Each URLFetcher gets an
- // id when it is created. The |url_fetcher_id_| is incremented by one for each
- // newly created URLFetcher. The URLFetcher ID can be used during testing to
- // get individual URLFetchers and modify their state. Outside of tests this ID
- // is not used.
- int next_url_fetcher_id_;
-
- DISALLOW_COPY_AND_ASSIGN(ImageDataFetcher);
-};
-
-} // namespace image_fetcher
-
-#endif // COMPONENTS_IMAGE_FETCHER_IMAGE_DATA_FETCHER_H_
« no previous file with comments | « components/image_fetcher/core/request_metadata_unittest.cc ('k') | components/image_fetcher/image_data_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698