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

chrome/common/extensions/extension_file_util.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_file_util.h"
 
 #include "app/l10n_util.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/scoped_temp_dir.h"
@@ skipping 390 matching lines @@
   return message_bundle;
 }
 
 static bool ValidateLocaleInfo(const Extension& extension, std::string* error) {
   // default_locale and _locales have to be both present or both missing.
   const FilePath path = extension.path().Append(Extension::kLocaleFolder);
   bool path_exists = file_util::PathExists(path);
   std::string default_locale = extension.default_locale();
 
   // If both default locale and _locales folder are empty, skip verification.
-  if (!default_locale.empty() || path_exists) {
-    if (default_locale.empty() && path_exists) {
-      *error = errors::kLocalesNoDefaultLocaleSpecified;
-      return false;
-    } else if (!default_locale.empty() && !path_exists) {
-      *error = errors::kLocalesTreeMissing;
+  if (default_locale.empty() && !path_exists)
+    return true;
+
+  if (default_locale.empty() && path_exists) {
+    *error = errors::kLocalesNoDefaultLocaleSpecified;
+    return false;
+  } else if (!default_locale.empty() && !path_exists) {
+    *error = errors::kLocalesTreeMissing;
+    return false;
+  }
+
+  // Treat all folders under _locales as valid locales.
+  file_util::FileEnumerator locales(path,
+                                    false,
+                                    file_util::FileEnumerator::DIRECTORIES);
+
+  std::set<std::string> all_locales;
+  extension_l10n_util::GetAllLocales(&all_locales);
+  const FilePath default_locale_path = path.AppendASCII(default_locale);
+  bool has_default_locale_message_file = false;
+
+  FilePath locale_path;
+  while (!(locale_path = locales.Next()).empty()) {
+    if (extension_l10n_util::ShouldSkipValidation(path, locale_path,
+                                                  all_locales))
+      continue;
+
+    FilePath messages_path =
+        locale_path.Append(Extension::kMessagesFilename);
+
+    if (!file_util::PathExists(messages_path)) {
+      *error = StringPrintf(
+          "%s %s", errors::kLocalesMessagesFileMissing,
+          WideToUTF8(messages_path.ToWStringHack()).c_str());
       return false;
     }
 
-    // Treat all folders under _locales as valid locales.
-    file_util::FileEnumerator locales(path,
-                                      false,
-                                      file_util::FileEnumerator::DIRECTORIES);
+    if (locale_path == default_locale_path)
+      has_default_locale_message_file = true;
+  }
 
-    FilePath locale_path = locales.Next();
-    if (locale_path.empty()) {
-      *error = errors::kLocalesTreeMissing;
-      return false;
-    }
-
-    const FilePath default_locale_path = path.AppendASCII(default_locale);
-    bool has_default_locale_message_file = false;
-    do {
-      // Skip any strings with '.'. This happens sometimes, for example with
-      // '.svn' directories.
-      FilePath relative_path;
-      if (!extension.path().AppendRelativePath(locale_path, &relative_path))
-        NOTREACHED();
-      std::wstring subdir(relative_path.ToWStringHack());
-      if (std::find(subdir.begin(), subdir.end(), L'.') != subdir.end())
-        continue;
-
-      FilePath messages_path =
-        locale_path.Append(Extension::kMessagesFilename);
-
-      if (!file_util::PathExists(messages_path)) {
-        *error = StringPrintf(
-            "%s %s", errors::kLocalesMessagesFileMissing,
-            WideToUTF8(messages_path.ToWStringHack()).c_str());
-        return false;
-      }
-
-      if (locale_path == default_locale_path)
-        has_default_locale_message_file = true;
-    } while (!(locale_path = locales.Next()).empty());
-
-    // Only message file for default locale has to exist.
-    if (!has_default_locale_message_file) {
-      *error = errors::kLocalesNoDefaultMessages;
-      return false;
-    }
+  // Only message file for default locale has to exist.
+  if (!has_default_locale_message_file) {
+    *error = errors::kLocalesNoDefaultMessages;
+    return false;
   }
 
   return true;
 }
 
 bool CheckForIllegalFilenames(const FilePath& extension_path,
                               std::string* error) {
   // Reserved underscore names.
   static const FilePath::CharType* reserved_names[] = {
     Extension::kLocaleFolder,
@@ skipping 40 matching lines @@
       UnescapeRule::SPACES | UnescapeRule::URL_SPECIAL_CHARS);
 
 #if defined(OS_POSIX)
   return FilePath(file_path);
 #elif defined(OS_WIN)
   return FilePath(UTF8ToWide(file_path));
 #endif
 }
 
 }  // namespace extension_file_util