ICU msg format support with more than one arguments

extensions/extensions.gyp

 # 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.
 
 {
   'includes': [
     'extensions.gypi',
   ],
   'variables': {
     'chromium_code': 1,
@@ skipping 105 matching lines @@
           ],
         }],
       ],
     },
     {
       # GN version: //extensions/browser
       'target_name': 'extensions_browser',
       'type': 'static_library',
       'dependencies': [
         '../base/base.gyp:base',
+        '../base/base.gyp:base_i18n',
+        '../third_party/icu/icu.gyp:icuuc',
+        '../third_party/icu/icu.gyp:icui18n',

[jungshik at Google, 2015/08/01 14:00:58]

Removing two icu lines above still works, but I'm not sure if it's 'canonical'
to do. 

[Ryan Sleevi, 2015/08/03 08:01:41]

Yeah, since none of the unittests explicitly use ICU, only the base::i18n parts,
it's entirely appropriate and matches your .gn changes (GYP's version of
public_deps is export_dependent_settings )

         '../base/base.gyp:base_prefs',
         '../components/components.gyp:browsing_data',
         '../components/components.gyp:device_event_log_component',
         '../components/components.gyp:guest_view_browser',
         '../components/components.gyp:keyed_service_content',
         '../components/components.gyp:keyed_service_core',
         '../components/components.gyp:onc_component',
         '../components/components.gyp:pref_registry',
         '../components/components.gyp:sessions_content',
         '../components/components.gyp:storage_monitor',
@@ skipping 174 matching lines @@
               '<(SHARED_INTERMEDIATE_DIR)/blink/devtools_resources.pak',
             ],
             'pak_output': '<(PRODUCT_DIR)/extensions_shell_and_test.pak',
           },
           'includes': [ '../build/repack_action.gypi' ],
         },
       ],
     },
   ]
 }