Addressing review comments that came after merging previous CL.
Minor updates to naming conventions and comments.
BUG=chromium-os:34299
TEST=Retested on ZGB. Re-ran unit tests.
Change-Id: I7db665d4f69969a972ee801f0e0cea9cf33437a6
Reviewed-on: https://gerrit.chromium.org/gerrit/36531
Reviewed-by: Gilad Arnold <garnold@chromium.org>
Commit-Ready: Jay Srinivasan <jaysri@chromium.org>
Reviewed-by: Jay Srinivasan <jaysri@chromium.org>
Tested-by: Jay Srinivasan <jaysri@chromium.org>
diff --git a/download_action.h b/download_action.h
index 98cde46..6e334f6 100644
--- a/download_action.h
+++ b/download_action.h
@@ -100,15 +100,18 @@
// Update Engine preference store.
PrefsInterface* prefs_;
+ // Global context for the system.
+ SystemState* system_state_;
+
+ // Pointer to the HttpFetcher that does the http work.
+ scoped_ptr<HttpFetcher> http_fetcher_;
+
// The FileWriter that downloaded data should be written to. It will
// either point to *decompressing_file_writer_ or *delta_performer_.
FileWriter* writer_;
scoped_ptr<DeltaPerformer> delta_performer_;
- // Pointer to the HttpFetcher that does the http work.
- scoped_ptr<HttpFetcher> http_fetcher_;
-
// Used by TransferTerminated to figure if this action terminated itself or
// was terminated by the action processor.
ActionExitCode code_;
@@ -117,9 +120,6 @@
DownloadActionDelegate* delegate_;
uint64_t bytes_received_;
- // Global context for the system.
- SystemState* system_state_;
-
DISALLOW_COPY_AND_ASSIGN(DownloadAction);
};