Use a more suitable storage for the timezone table.

base/i18n/timezone.cc

 // Copyright 2013 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 "base/i18n/timezone.h"
 
 #include <map>
 
 #include "base/memory/singleton.h"
 #include "base/strings/string16.h"
@@ skipping 17 matching lines @@
 
     return std::string();
   }
 
  private:
   TimezoneMap() {
     // These mappings are adapted from zone.tab, which is available at
     // <http://www.ietf.org/timezones/data/zone.tab> and is a part of public
     // domain.
     struct OlsonCodeData {
-      std::string country_code;
-      std::string olson_code;
-    } olson_code_data[] = {
+      const char* country_code;
+      const char* olson_code;
+    };
+    static const OlsonCodeData olson_code_data[] = {
         { "AD", "Europe/Andorra" },
         { "AE", "Asia/Dubai" },
         { "AF", "Asia/Kabul" },
         { "AG", "America/Antigua" },
         { "AI", "America/Anguilla" },
         { "AL", "Europe/Tirane" },
         { "AM", "Asia/Yerevan" },
         { "AO", "Africa/Luanda" },
         { "AQ", "Antarctica/McMurdo" },
         { "AQ", "Antarctica/Rothera" },
@@ skipping 401 matching lines @@
         { "YT", "Indian/Mayotte" },
         { "ZA", "Africa/Johannesburg" },
         { "ZM", "Africa/Lusaka" },
         { "ZW", "Africa/Harare" },
         // The mappings below are custom additions to zone.tab.
         { "GB", "Etc/GMT" },
         { "GB", "Etc/UTC" },
         { "GB", "Etc/UCT" },
     };
 
     for (size_t i = 0; i < ARRAYSIZE_UNSAFE(olson_code_data); ++i) {

[Mark Mentovai, 2014/10/01 13:05:07]

This can be normal arraysize now. Same below.

[Daniel Bratell, 2014/10/01 19:56:22]

Done.

-      map_[olson_code_data[i].olson_code] = olson_code_data[i].country_code;
+      map_[std::string(olson_code_data[i].olson_code)] =
+        std::string(olson_code_data[i].country_code);

[Mark Mentovai, 2014/10/01 13:05:07]

You can make the “value” side of the map store const char* instead of
std::string at least, and avoid these constructions.

If you want to fiddle with a comparator template argument, you can also make the
“key” side use const char*.

You may also be able to bubble these changes through to CountryCodeForTimezone()
and CountryCodeForCurrentTimezone().

[Daniel Bratell, 2014/10/01 19:56:22]

Done.

     }
 
     // These are mapping from old codenames to new codenames. They are also
     // part of public domain, and available at
     // <http://www.ietf.org/timezones/data/backward>.
     struct LinkData {
-      std::string old_code;
-      std::string new_code;
-    } link_data[] = {
+      const char* old_code;
+      const char* new_code;
+    };
+    static const LinkData link_data[] = {
         { "Africa/Asmera", "Africa/Asmara" },
         { "Africa/Timbuktu", "Africa/Bamako" },
         { "America/Argentina/ComodRivadavia", "America/Argentina/Catamarca" },
         { "America/Atka", "America/Adak" },
         { "America/Buenos_Aires", "America/Argentina/Buenos_Aires" },
         { "America/Catamarca", "America/Argentina/Catamarca" },
         { "America/Coral_Harbour", "America/Atikokan" },
         { "America/Cordoba", "America/Argentina/Cordoba" },
         { "America/Ensenada", "America/Tijuana" },
         { "America/Fort_Wayne", "America/Indiana/Indianapolis" },
@@ skipping 93 matching lines @@
         { "US/Mountain", "America/Denver" },
         { "US/Pacific", "America/Los_Angeles" },
         { "US/Samoa", "Pacific/Pago_Pago" },
         { "UTC", "Etc/UTC" },
         { "Universal", "Etc/UTC" },
         { "W-SU", "Europe/Moscow" },
         { "Zulu", "Etc/UTC" },
     };
 
     for (size_t i = 0; i < ARRAYSIZE_UNSAFE(link_data); ++i) {
-      map_[link_data[i].old_code] = map_[link_data[i].new_code];
+      map_[std::string(link_data[i].old_code)] =
+        map_[std::string(link_data[i].new_code)];
     }
   }
 
   friend struct DefaultSingletonTraits<TimezoneMap>;
 
   std::map<std::string, std::string> map_;
 
   DISALLOW_COPY_AND_ASSIGN(TimezoneMap);
 };
 
 }  // namespace
 
 std::string CountryCodeForCurrentTimezone() {
   scoped_ptr<icu::TimeZone> zone(icu::TimeZone::createDefault());
   icu::UnicodeString id;
   zone->getID(id);
   string16 olson_code(id.getBuffer(), id.length());
   return TimezoneMap::GetInstance()->CountryCodeForTimezone(
       UTF16ToUTF8(olson_code));
 }
 
 }  // namespace base