Consolidate the update_client serialization code.

components/update_client/component.h

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_UPDATE_CLIENT_COMPONENT_H_
 #define COMPONENTS_UPDATE_CLIENT_COMPONENT_H_
 
 #include <map>
 #include <memory>
 #include <string>
@@ skipping 46 matching lines @@
   void UpdateCheckComplete() const;
 
   // Returns true if the component has reached a final state and no further
   // handling and state transitions are possible.
   bool IsHandled() const { return state_->IsFinal(); }
 
   // Returns true if an update is available for this component, meaning that
   // the update server has return a response containing an update.
   bool IsUpdateAvailable() const { return is_update_available_; }
 
-  // Returns true if a ping must be sent back to the server. As a general rule,
-  // a ping is sent only for server responses containing instructions to update.
-  bool CanPing() const {
-    return IsUpdateAvailable() || state() == ComponentState::kUninstalled;
-  }
-
+  /*
+    // Returns true if a ping must be sent back to the server. As a general
+    rule,
+    // a ping is sent only for server responses containing instructions to
+    update. bool CanPing() const { return IsUpdateAvailable() || state() ==
+    ComponentState::kUninstalled;
+    }
+  */
   base::TimeDelta GetUpdateDuration() const;
 
   ComponentState state() const { return state_->state(); }
 
   std::string id() const { return id_; }
 
   const CrxComponent& crx_component() const { return crx_component_; }
   void set_crx_component(const CrxComponent& crx_component) {
     crx_component_ = crx_component;
   }
@@ skipping 19 matching lines @@
   }
 
   // Returns the time when processing of an update for this component has
   // begun, once the update has been discovered. Returns a null TimeTicks object
   // if the handling of an update has not happened.
   // base::TimeTicks update_begin() const { return update_begin_; }
 
   bool on_demand() const { return on_demand_; }
   void set_on_demand(bool on_demand) { on_demand_ = on_demand; }
 
-  const std::vector<CrxDownloader::DownloadMetrics>& download_metrics() const {
-    return download_metrics_;
-  }
+  const std::vector<std::string>& events() const { return events_; }
 
   const std::vector<GURL>& crx_diffurls() const { return crx_diffurls_; }
 
   bool diff_update_failed() const { return !!diff_error_code_; }
 
   int error_category() const { return error_category_; }
   int error_code() const { return error_code_; }
   int extra_code1() const { return extra_code1_; }
   int diff_error_category() const { return diff_error_category_; }
   int diff_error_code() const { return diff_error_code_; }
@@ skipping 215 matching lines @@
     // State overrides.
     void DoHandle() override;
 
     DISALLOW_COPY_AND_ASSIGN(StateUninstalled);
   };
 
   // Returns true is the update payload for this component can be downloaded
   // by a downloader which can do bandwidth throttling on the client side.
   bool CanDoBackgroundDownload() const;
 
-  void AppendDownloadMetrics(
-      const std::vector<CrxDownloader::DownloadMetrics>& download_metrics);
+  void AppendEvent(const std::string& event);
 
   // Changes the component state and notifies the caller of the |Handle|
   // function that the handling of this component state is complete.
   void ChangeState(std::unique_ptr<State> next_state);
 
   // Notifies registered observers about changes in the state of the component.
   void NotifyObservers(Events event) const;
 
   base::ThreadChecker thread_checker_;
 
@@ skipping 42 matching lines @@
   // the |extra_code1| usually contains a system error, but it can contain
   // any extended information that is relevant to either the category or the
   // error itself.
   int error_category_ = 0;
   int error_code_ = 0;
   int extra_code1_ = 0;
   int diff_error_category_ = 0;
   int diff_error_code_ = 0;
   int diff_extra_code1_ = 0;
 
-  std::vector<CrxDownloader::DownloadMetrics> download_metrics_;
+  std::vector<std::string> events_;
 
   CallbackHandleComplete callback_handle_complete_;
   std::unique_ptr<State> state_;
   const UpdateContext& update_context_;
 
   base::Closure update_check_complete_;
 
   DISALLOW_COPY_AND_ASSIGN(Component);
 };
 
 using IdToComponentPtrMap = std::map<std::string, std::unique_ptr<Component>>;
 
 }  // namespace update_client
 
 #endif  // COMPONENTS_UPDATE_CLIENT_COMPONENT_H_