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

Side by Side Diff: components/image_fetcher/core/image_fetcher_impl.cc

Issue 2781473003: Add |SetImageDownloadLimit| to ImageFetcher to limit downloaded bytes (Closed)
Patch Set: Test fix: Mock pure virtual Setter in CrOS notification test 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/image_fetcher/core/image_fetcher_impl.h" 5 #include "components/image_fetcher/core/image_fetcher_impl.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "net/base/load_flags.h" 10 #include "net/base/load_flags.h"
(...skipping 25 matching lines...) Expand all
36 36
37 void ImageFetcherImpl::SetDataUseServiceName( 37 void ImageFetcherImpl::SetDataUseServiceName(
38 DataUseServiceName data_use_service_name) { 38 DataUseServiceName data_use_service_name) {
39 image_data_fetcher_->SetDataUseServiceName(data_use_service_name); 39 image_data_fetcher_->SetDataUseServiceName(data_use_service_name);
40 } 40 }
41 41
42 void ImageFetcherImpl::SetDesiredImageFrameSize(const gfx::Size& size) { 42 void ImageFetcherImpl::SetDesiredImageFrameSize(const gfx::Size& size) {
43 desired_image_frame_size_ = size; 43 desired_image_frame_size_ = size;
44 } 44 }
45 45
46 void ImageFetcherImpl::SetImageDownloadLimit(
47 base::Optional<int64_t> max_download_bytes) {
48 image_data_fetcher_->SetImageDownloadLimit(max_download_bytes);
49 }
50
46 void ImageFetcherImpl::StartOrQueueNetworkRequest( 51 void ImageFetcherImpl::StartOrQueueNetworkRequest(
47 const std::string& id, 52 const std::string& id,
48 const GURL& image_url, 53 const GURL& image_url,
49 const ImageFetcherCallback& callback) { 54 const ImageFetcherCallback& callback) {
50 // Before starting to fetch the image. Look for a request in progress for 55 // Before starting to fetch the image. Look for a request in progress for
51 // |image_url|, and queue if appropriate. 56 // |image_url|, and queue if appropriate.
52 ImageRequestMap::iterator it = pending_net_requests_.find(image_url); 57 ImageRequestMap::iterator it = pending_net_requests_.find(image_url);
53 if (it == pending_net_requests_.end()) { 58 if (it == pending_net_requests_.end()) {
54 ImageRequest request; 59 ImageRequest request;
55 request.id = id; 60 request.id = id;
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
101 106
102 // Erase the completed ImageRequest. 107 // Erase the completed ImageRequest.
103 pending_net_requests_.erase(image_iter); 108 pending_net_requests_.erase(image_iter);
104 } 109 }
105 110
106 ImageDecoder* ImageFetcherImpl::GetImageDecoder() { 111 ImageDecoder* ImageFetcherImpl::GetImageDecoder() {
107 return image_decoder_.get(); 112 return image_decoder_.get();
108 } 113 }
109 114
110 } // namespace image_fetcher 115 } // namespace image_fetcher
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698