Simple fix of extension_l10n_util that allows folders in form ".some_name" to...

Simple fix of extension_l10n_util that allows folders in form ".some_name" to exist in _locales folder.
It helps testing/loading extensions from svn tree (skips .svn folder, doesn't fail).

[Nebojša Ćirić, 2009-09-04 20:38:54]

This is a fix for the other CL you are already reviewing (so I can simplify it).
It's very short, and I hope to land it by the end of the day :).

Cira

[Nebojša Ćirić, 2009-09-04 20:39:50]

We already talked about doing this (for easier testing). It's a prequel for the
other CL that loads the catalogs.

[Aaron Boodman, 2009-09-09 01:42:56]

lgtm

http://codereview.chromium.org/196029/diff/2001/2005
File chrome/browser/extensions/extension_l10n_util.cc (right):

http://codereview.chromium.org/196029/diff/2001/2005#newcode42
Line 42: if (locale_name->find(".", 0) == 0) {
Nit: braces not needed.

http://codereview.chromium.org/196029/diff/2001/2005#newcode42
Line 42: if (locale_name->find(".", 0) == 0) {
Nit: second parameter unnecessary as 0 is the default values.

[Nebojša Ćirić, 2009-09-09 20:06:45]

All done, thanks.

On 2009/09/09 01:42:56, Aaron Boodman wrote:
> lgtm
> 
> http://codereview.chromium.org/196029/diff/2001/2005
> File chrome/browser/extensions/extension_l10n_util.cc (right):
> 
> http://codereview.chromium.org/196029/diff/2001/2005#newcode42
> Line 42: if (locale_name->find(".", 0) == 0) {
> Nit: braces not needed.
> 
> http://codereview.chromium.org/196029/diff/2001/2005#newcode42
> Line 42: if (locale_name->find(".", 0) == 0) {
> Nit: second parameter unnecessary as 0 is the default values.