Removed wchar_t from Time::FromString

chrome/browser/web_resource/promo_resource_service.cc

 // Copyright (c) 2011 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 "chrome/browser/web_resource/promo_resource_service.h"
 
 #include "base/string_number_conversions.h"
 #include "base/threading/thread_restrictions.h"
 #include "base/time.h"
 #include "base/values.h"
@@ skipping 251 matching lines @@
           }
         }
       }
       if (!promo_start_string.empty() &&
           promo_start_string.length() > 0 &&
           !promo_end_string.empty() &&
           promo_end_string.length() > 0) {
         base::Time start_time;
         base::Time end_time;
         if (base::Time::FromString(
-                ASCIIToWide(promo_start_string).c_str(), &start_time) &&
+                promo_start_string.c_str(), &start_time) &&
             base::Time::FromString(
-                ASCIIToWide(promo_end_string).c_str(), &end_time)) {
+                promo_end_string.c_str(), &end_time)) {
           // Add group time slice, adjusted from hours to seconds.
           promo_start = start_time.ToDoubleT() +
               (prefs_->FindPreference(prefs::kNTPPromoGroup) ?
                   prefs_->GetInteger(prefs::kNTPPromoGroup) *
                       time_slice_hrs * 60 * 60 : 0);
           promo_end = end_time.ToDoubleT();
         }
       }
     }
   }
@@ skipping 139 matching lines @@
           }
         }
       }
       if (!logo_start_string.empty() &&
           logo_start_string.length() > 0 &&
           !logo_end_string.empty() &&
           logo_end_string.length() > 0) {
         base::Time start_time;
         base::Time end_time;
         if (base::Time::FromString(
-                ASCIIToWide(logo_start_string).c_str(), &start_time) &&
+                logo_start_string.c_str(), &start_time) &&
             base::Time::FromString(
-                ASCIIToWide(logo_end_string).c_str(), &end_time)) {
+                logo_end_string.c_str(), &end_time)) {

[wtc, 2011/07/13 17:01:31]

The arguments to the four base::Time::FromString calls in
this file may fit on the same line now.  Please give that a
try.

[shinyak (Google), 2011/07/13 23:28:33]

Done.

           logo_start = start_time.ToDoubleT();
           logo_end = end_time.ToDoubleT();
         }
       }
     }
   }
 
   // If logo start or end times have changed, trigger a new web resource
   // notification, so that the logo on the NTP is updated. This check is
   // outside the reading of the web resource data, because the absence of
@@ skipping 29 matching lines @@
     base::ThreadRestrictions::ScopedAllowIO allow_io;
     chrome::VersionInfo::Channel channel = chrome::VersionInfo::GetChannel();
     is_promo_build = PromoResourceService::IsBuildTargeted(
         channel, prefs->GetInteger(prefs::kNTPPromoBuild));
   }
 
   return !promo_closed && !is_synced && is_promo_build;
 }
 
 }  // namespace PromoResourceServiceUtil