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

Unified Diff: update_attempter.cc

Issue 5835004: AU: MultiHttpFetcher cleanup/rewrite (Closed) Base URL: http://git.chromium.org/git/update_engine.git@master
Patch Set: Created 10 years 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
« multi_range_http_fetcher.cc ('K') | « multi_range_http_fetcher.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: update_attempter.cc
diff --git a/update_attempter.cc b/update_attempter.cc
index 85500e61e65e6c4dafc027a7d3113165030a0f8a..d4769ed25ed384d08fba51606e76fff733b15b89 100644
--- a/update_attempter.cc
+++ b/update_attempter.cc
@@ -22,7 +22,7 @@
#include "update_engine/download_action.h"
#include "update_engine/filesystem_copier_action.h"
#include "update_engine/libcurl_http_fetcher.h"
-#include "update_engine/multi_http_fetcher.h"
+#include "update_engine/multi_range_http_fetcher.h"
#include "update_engine/omaha_request_action.h"
#include "update_engine/omaha_request_params.h"
#include "update_engine/omaha_response_handler_action.h"
@@ -185,8 +185,8 @@ void UpdateAttempter::Update(const std::string& app_version,
OmahaEvent::kTypeUpdateDownloadStarted),
new LibcurlHttpFetcher(GetProxyResolver())));
shared_ptr<DownloadAction> download_action(
- new DownloadAction(prefs_, new MultiHttpFetcher<LibcurlHttpFetcher>(
- GetProxyResolver())));
+ new DownloadAction(prefs_, new MultiRangeHTTPFetcher(
+ new LibcurlHttpFetcher(GetProxyResolver()))));
shared_ptr<OmahaRequestAction> download_finished_action(
new OmahaRequestAction(prefs_,
omaha_request_params_,
@@ -554,10 +554,9 @@ void UpdateAttempter::MarkDeltaUpdateFailure() {
}
void UpdateAttempter::SetupDownload() {
- MultiHttpFetcher<LibcurlHttpFetcher>* fetcher =
- dynamic_cast<MultiHttpFetcher<LibcurlHttpFetcher>*>(
- download_action_->http_fetcher());
- MultiHttpFetcher<LibcurlHttpFetcher>::RangesVect ranges;
+ MultiRangeHTTPFetcher* fetcher =
+ dynamic_cast<MultiRangeHTTPFetcher*>(download_action_->http_fetcher());
+ MultiRangeHTTPFetcher::RangesVect ranges;
if (response_handler_action_->install_plan().is_resume) {
// Resuming an update so fetch the update manifest metadata first.
int64_t manifest_metadata_size = 0;
« multi_range_http_fetcher.cc ('K') | « multi_range_http_fetcher.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698