Move remote_db_manager into the safe_browsing_db component.

components/components_tests.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.
 
 {
   'variables': {
     # This turns on e.g. the filename-based detection of which
     # platforms to include source files on (e.g. files ending in
     # _mac.h or _mac.cc are only compiled on MacOSX).
     'chromium_code': 1,
@@ skipping 1242 matching lines @@
                     '../build/linux/system.gyp:ssl',
                   ],
                 }, {
                   'dependencies': [
                     '../third_party/nss/nss.gyp:nspr',
                     '../third_party/nss/nss.gyp:nss',
                   ],
                 }],
               ],
             }],
+            ['safe_browsing == 2 and OS != "ios"', {
+              'dependencies': [
+                'components.gyp:safe_browsing_db_mobile',
+              ],
+              'sources': [
+                'safe_browsing_db/remote_database_manager_unittest.cc',

[Nathan Parker, 2016/01/20 23:26:54]

Can we specify a target here rather than individual sources? I do see a lot of
component/*.cc's above, but I'm not clear why those are listed in the component
instead. 

[vakh (use Gerrit instead), 2016/01/21 01:36:46]

Not sure if we can specify targets for sources. The closest thing is a group,
such as: '<@(nacl_unittest_sources)'
However, for just 2 files, it probably isn't that useful.

[blundell, 2016/01/21 08:51:27]

I think it would be more in the spirit of how things are generally organized in
this file to define a variable for these unittests.

[vakh (use Gerrit instead), 2016/01/21 18:48:46]

Done.

+                'safe_browsing_db/safe_browsing_api_handler_unittest.cc',
+              ],
+            }],
           ],
         }, { # 'OS == "ios"'
           'sources': [
             'webp_transcode/webp_decoder_unittest.mm',
             'webp_transcode/webp_network_client_unittest.mm',
           ],
           'sources/': [
             # Exclude all tests that depends on //content (based on layered-
             # component directory structure).
             ['exclude', '^[^/]*/content/'],
@@ skipping 610 matching lines @@
               'dependencies': [
                 '../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
               ],
             }],
           ],
         },
       ],
     }],
   ],
 }