Remove NO_LOG LogType enum value.

components/metrics/metrics_log_manager.cc

 // Copyright 2014 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.
 
 #include "components/metrics/metrics_log_manager.h"
 
 #include <algorithm>
 
 #include "base/metrics/histogram.h"
 #include "base/strings/string_util.h"
@@ skipping 84 matching lines @@
   DCHECK(!paused_log_.get());
   paused_log_.reset(current_log_.release());
 }
 
 void MetricsLogManager::ResumePausedLog() {
   DCHECK(!current_log_.get());
   current_log_.reset(paused_log_.release());
 }
 
 void MetricsLogManager::StoreLog(std::string* log, LogType log_type) {
-  DCHECK_NE(MetricsLogBase::NO_LOG, log_type);
-  metrics::PersistedLogs* destination_queue =
-      (log_type == MetricsLogBase::INITIAL_STABILITY_LOG) ?
-      &initial_log_queue_ : &ongoing_log_queue_;
-
-  destination_queue->StoreLog(log);
+  switch (log_type) {
+    case MetricsLogBase::INITIAL_STABILITY_LOG:
+      initial_log_queue_.StoreLog(log);
+      break;
+    case MetricsLogBase::ONGOING_LOG:
+      ongoing_log_queue_.StoreLog(log);

[Ilya Sherman, 2014/06/02 22:06:19]

nit: Probably wise to have a "break;" stmt here as well, in case anyone ever
adds to the switch stmt.

[Alexei Svitkine (slow), 2014/06/03 16:52:53]

Done.

+  }
 }
 
 void MetricsLogManager::StoreStagedLogAsUnsent(
     metrics::PersistedLogs::StoreType store_type) {
   DCHECK(has_staged_log());
   if (initial_log_queue_.has_staged_log())
     initial_log_queue_.StoreStagedLogAsUnsent(store_type);
   else
     ongoing_log_queue_.StoreStagedLogAsUnsent(store_type);
 }
@@ skipping 19 matching lines @@
 void MetricsLogManager::LoadPersistedUnsentLogs() {
   base::ElapsedTimer timer;
   initial_log_queue_.DeserializeLogs();
   ongoing_log_queue_.DeserializeLogs();
   UMA_HISTOGRAM_TIMES("UMA.LoadLogsTime", timer.Elapsed());
 
   unsent_logs_loaded_ = true;
 }
 
 } // namespace metrics