commit edce28385c15d953bd04f7bc715c45f02dc850f6
author Jay Srinivasan <jaysri@chromium.org> Wed Oct 24 18:57:47 2012 -0700
committer Gerrit <chrome-bot@google.com> Thu Oct 25 12:06:28 2012 -0700
tree 80fc02181cad04243e9b748f3ece5018daad203d
parent f057205065339c54d3403263f049e2ea69417665

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>

download_action.cc

diff --git a/download_action.cc b/download_action.cc
index b2055c8..8aa9e35 100644
--- a/download_action.cc
+++ b/download_action.cc
@@ -24,12 +24,12 @@
                                SystemState* system_state,
                                HttpFetcher* http_fetcher)
     : prefs_(prefs),
-      writer_(NULL),
+      system_state_(system_state),
       http_fetcher_(http_fetcher),
+      writer_(NULL),
       code_(kActionCodeSuccess),
       delegate_(NULL),
-      bytes_received_(0),
-      system_state_(system_state) {}
+      bytes_received_(0) {}
 
 DownloadAction::~DownloadAction() {}