Remove NO_LOG LogType enum value.

Remove NO_LOG LogType enum value.

Also, changes a condition to a switch to
make compiler warn if a new enum value is
added.

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274582

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

LGTM % nit.  Thanks!

https://codereview.chromium.org/310483009/diff/20001/components/metrics/metri...
File components/metrics/metrics_log_manager.cc (right):

https://codereview.chromium.org/310483009/diff/20001/components/metrics/metri...
components/metrics/metrics_log_manager.cc:110: ongoing_log_queue_.StoreLog(log);
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]

https://codereview.chromium.org/310483009/diff/20001/components/metrics/metri...
File components/metrics/metrics_log_manager.cc (right):

https://codereview.chromium.org/310483009/diff/20001/components/metrics/metri...
components/metrics/metrics_log_manager.cc:110: ongoing_log_queue_.StoreLog(log);
On 2014/06/02 22:06:19, Ilya Sherman wrote:
> nit: Probably wise to have a "break;" stmt here as well, in case anyone ever
> adds to the switch stmt.

Done.

[Alexei Svitkine (slow), 2014-06-03 16:52:56]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2014-06-03 16:53:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/asvitkine@chromium.org/310483009/40001

[commit-bot: I haz the power, 2014-06-03 19:05:36]

Change committed as 274582