Remove base_i18n dependency from //components/metrics:metrics target

components/metrics/daily_event.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/daily_event.h"
 
 #include <utility>
 
-#include "base/i18n/time_formatting.h"
 #include "base/metrics/histogram.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 
 namespace metrics {
 
 namespace {
 
 enum IntervalType {
   FIRST_RUN,
@@ skipping 42 matching lines @@
   DCHECK(last_fired_.is_null());
   observers_.push_back(std::move(observer));
 }
 
 void DailyEvent::CheckInterval() {
   base::Time now = base::Time::Now();
   if (last_fired_.is_null()) {
     // The first time we call CheckInterval, we read the time stored in prefs.
     last_fired_ = base::Time::FromInternalValue(
         pref_service_->GetInt64(pref_name_));
-    DVLOG(1) << "DailyEvent time loaded: "
-             << base::TimeFormatShortDateAndTime(last_fired_);
+    DVLOG(1) << "DailyEvent time loaded: " << last_fired_;

[xunjieli, 2016/03/21 21:02:13]

base::Time converts to a UTC timestamp automatically.  Since it is only used in
debug logging, I guess it is fine ?

     if (last_fired_.is_null()) {
       DVLOG(1) << "DailyEvent first run.";
       RecordIntervalTypeHistogram(histogram_name_, FIRST_RUN);
       OnInterval(now);
       return;
     }
   }
   int days_elapsed = (now - last_fired_).InDays();
   if (days_elapsed >= 1) {
     DVLOG(1) << "DailyEvent day elapsed.";
@@ skipping 15 matching lines @@
 
   // Notify all observers
   for (ScopedVector<DailyEvent::Observer>::iterator it = observers_.begin();
        it != observers_.end();
        ++it) {
     (*it)->OnDailyEvent();
   }
 }
 
 }  // namespace metrics