New thin layer of API extension chrome.i18n.detectLanguage

chrome/browser/extensions/api/api_registration.gyp

 # Copyright 2014 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.
 
 {
   'targets': [
     {
       # GN version: //chrome/browser/extensions/api:api_registration
       'target_name': 'chrome_api_registration',
       'type': 'static_library',
@@ skipping 17 matching lines @@
         '<(DEPTH)/skia/skia.gyp:skia',
         '<(DEPTH)/sync/sync.gyp:sync',
         '<(DEPTH)/ui/accessibility/accessibility.gyp:ax_gen',
       ],
       'conditions': [
         ['chromeos==1', {
           'dependencies': [
             '<(DEPTH)/chrome/chrome.gyp:drive_proto',
           ],
         }],
+        [ 'cld_version==0 or cld_version==2', {

[Ken Rockot(use gerrit already), 2015/07/21 22:28:39]

nit: no need for the space after [ 

[amalika, 2015/07/21 22:40:56]

Done.

+          'dependencies': [
+            # Interactive tests should use whatever CLD2 data access mode that
+            # the application embedder is using.
+            '<(DEPTH)/third_party/cld_2/cld_2.gyp:cld2_platform_impl', ],
+        }],

[Ken Rockot(use gerrit already), 2015/07/21 22:28:39]

i don't really know anything about CLD. I wonder if it's also a good idea to add
a clause for version 1, something like this:

    ['cld_version==1', {
      'dependencies': [
        '<(DEPTH)/third_party/cld/cld.gyp:cld',
      },
    }],

[amalika, 2015/07/21 22:40:56]

As of November 20, 2014 the sole remaining platform using CLD1 is Android. Our
API is only available on desktop. 
In i18n.cc file we also only have support for CLD2 as per reason above, so I
guess it would be unnecessary to add a dependency for CLD1?

       ],
     },
   ],
 }