All platforms don't support same locales at the same time. Make locale check ...

chrome/common/extensions/extension_unpacker.cc

 // Copyright (c) 2009 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/common/extensions/extension_unpacker.h"
 
 #include "base/file_util.h"
 #include "base/scoped_handle.h"
 #include "base/scoped_temp_dir.h"
 #include "base/string_util.h"
 #include "base/thread.h"
 #include "base/values.h"
 #include "net/base/file_stream.h"
 #include "chrome/common/common_param_traits.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/extension_file_util.h"
+#include "chrome/common/extensions/extension_l10n_util.h"
 #include "chrome/common/json_value_serializer.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/common/zip.h"
 #include "ipc/ipc_message_utils.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "webkit/glue/image_decoder.h"
 
 namespace errors = extension_manifest_errors;
 namespace keys = extension_manifest_keys;
@@ skipping 80 matching lines @@
   }
 
   return static_cast<DictionaryValue*>(root.release());
 }
 
 bool ExtensionUnpacker::ReadAllMessageCatalogs(
     const std::string& default_locale) {
   FilePath locales_path =
     temp_install_dir_.Append(Extension::kLocaleFolder);
 
-  // Treat all folders under _locales as valid locales.
+  // Not all folders under _locales are valid locales.
   file_util::FileEnumerator locales(locales_path,
                                     false,
                                     file_util::FileEnumerator::DIRECTORIES);
 
-  FilePath locale_path = locales.Next();
-  do {
-    // Since we use this string as a key in a DictionaryValue, be paranoid about
-    // skipping any strings with '.'. This happens sometimes, for example with
-    // '.svn' directories.
-    FilePath relative_path;
-    // message_path was created from temp_install_dir. This should never fail.
-    if (!temp_install_dir_.AppendRelativePath(locale_path, &relative_path))
-      NOTREACHED();
-    std::wstring subdir(relative_path.ToWStringHack());
-    if (std::find(subdir.begin(), subdir.end(), L'.') != subdir.end())
+  std::set<std::string> all_locales;
+  extension_l10n_util::GetAllLocales(&all_locales);
+  FilePath locale_path;
+  while (!(locale_path = locales.Next()).empty()) {
+    if (extension_l10n_util::ShouldSkipValidation(locales_path, locale_path,
+                                                  all_locales))
       continue;
 
     FilePath messages_path =
       locale_path.Append(Extension::kMessagesFilename);
 
     if (!ReadMessageCatalog(messages_path))
       return false;
-  } while (!(locale_path = locales.Next()).empty());
+  }
 
   return true;
 }
 
 bool ExtensionUnpacker::Run() {
   LOG(INFO) << "Installing extension " << extension_path_.value();
 
   // <profile>/Extensions/INSTALL_TEMP/<version>
   temp_install_dir_ =
       extension_path_.DirName().AppendASCII(kTempExtensionName);
@@ skipping 115 matching lines @@
     NOTREACHED();
 
   parsed_catalogs_->Set(relative_path.DirName().ToWStringHack(), root);
 
   return true;
 }
 
 void ExtensionUnpacker::SetError(const std::string &error) {
   error_message_ = error;
 }