[iOS] Pack ios_strings_resources_${locale}.pak files

chrome/chrome_resources.gyp

Index: chrome/chrome_resources.gyp
diff --git a/chrome/chrome_resources.gyp b/chrome/chrome_resources.gyp
index 49018c56e28722a3eebfa71d43c47e3793fc5478..34a5d3946fae0e38747098840a1ec3294f220179 100644
--- a/chrome/chrome_resources.gyp
+++ b/chrome/chrome_resources.gyp
@@ -452,14 +452,14 @@
             '<(DEPTH)/content/app/strings/content_strings.gyp:content_strings',
             '<(DEPTH)/third_party/WebKit/public/blink_resources.gyp:blink_resources',
           ],
-        }, {  # else

[Nico, 2015/04/28 18:07:34]

why this change?

[sdefresne, 2015/04/29 07:36:24]

When editing gyp files, I find "else" block in condition really hard to find.
Here I was looking for 'OS == "ios"' block and could not find it so added a new
one, and then later found this "else" and merged them.

BTW, I'm probably not the only one to find "else" hard to understand since the
original code retested the same condition (that second test was useless) inside
the else block as you can see below. I think that removing the else and adding a
new 'OS == "ios"' block explicitly make it the code easier to read and update.

[Nico, 2015/04/29 15:33:16]

I find the new spelling very confusing. Please keep this as it was (in this
change, not in a follow-up).

[sdefresne, 2015/04/29 16:26:58]

Sure. Done.

+        }],
+        ['OS == "ios"', {
+          'dependencies': [
+            '<(DEPTH)/ios/chrome/ios_chrome_resources.gyp:ios_strings_resources_gen',
+          ],
           'actions': [
             {
-              'conditions': [
-                ['OS == "ios"', {
-                  'includes': ['chrome_repack_chrome_300_percent.gypi']
-                }],
-              ],
+              'includes': ['chrome_repack_chrome_300_percent.gypi']
             },
           ],
         }],