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

Unified Diff: src/platform/update_engine/update_check_action.cc

Issue 2044001: AU: Minor fixes to get it to do full update on real device (Closed) Base URL: ssh://git@chromiumos-git/chromeos
Patch Set: fixes for review Created 10 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
« no previous file with comments | « src/platform/update_engine/update_attempter.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/platform/update_engine/update_check_action.cc
diff --git a/src/platform/update_engine/update_check_action.cc b/src/platform/update_engine/update_check_action.cc
index 106400b90c280d57bac8ffe4ff60f2b3c9506e45..57e82eb1c43e1ba9cb4475c5aa80e81813f8285b 100644
--- a/src/platform/update_engine/update_check_action.cc
+++ b/src/platform/update_engine/update_check_action.cc
@@ -107,6 +107,8 @@ void UpdateCheckAction::PerformAction() {
http_fetcher_->set_delegate(this);
string request_post(FormatRequest(params_));
http_fetcher_->SetPostData(request_post.data(), request_post.size());
+ LOG(INFO) << "Checking for update at " << params_.update_url;
+ LOG(INFO) << "Request: " << request_post;
http_fetcher_->BeginTransfer(params_.update_url);
}
@@ -190,8 +192,12 @@ off_t ParseInt(const string& str) {
void UpdateCheckAction::TransferComplete(HttpFetcher *fetcher,
bool successful) {
ScopedActionCompleter completer(processor_, this);
- if (!successful)
+ LOG(INFO) << "Update check response: " << string(response_buffer_.begin(),
+ response_buffer_.end());
+ if (!successful) {
+ LOG(ERROR) << "Update check network transfer failed.";
return;
+ }
if (!HasOutputPipe()) {
// Just set success to whether or not the http transfer succeeded,
// which must be true at this point in the code.
« no previous file with comments | « src/platform/update_engine/update_attempter.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698