Split to SafeBrowsingDatabaseManager into Local* and Remote*.

chrome/chrome_browser.gypi

 # Copyright 2012 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': {
     # All .cc, .h, .m, and .mm files under browser except for:
     # * tests and mocks.
     # * code below browser/chromeos
     # * code below browser/extensions
@@ skipping 2336 matching lines @@
       'browser/profiles/storage_partition_descriptor.h',
     ],
     'chrome_browser_rlz_sources': [
       'browser/rlz/rlz.cc',
       'browser/rlz/rlz.h',
       'browser/rlz/rlz_chromeos.cc',
       'browser/rlz/rlz_ios.mm',
       'browser/rlz/rlz_mac.cc',
       'browser/rlz/rlz_win.cc',
     ],
-    # Used for safe browsing in basic mode (safe_browsing=2) as well as full
-    # mode (safe_browsing=1).
+    # "Safe Browsing Basic" files used for safe browsing in full mode
+    # (safe_browsing=1), mobile (=2), and mobile-extended (=3).
     'chrome_browser_safe_browsing_basic_sources': [
+      'browser/renderer_host/safe_browsing_resource_throttle.cc',

[mattm, 2015/05/08 02:17:26]

I thought this still shouldn't be used for mobile(=2)?

[Nathan Parker, 2015/05/12 01:04:01]

Yes there are a few extra sources here that aren't needed for mode=2 (I expect
the compiler will remove the .objs that have no use).  To get rid them here,
I'll add them them in both mobile_extended_sources and full_sources.  I was
trying to avoid that duplication since it can lead to mistakes.

(The alternative is a fourth list,
'chrome_browser_safe_browsing_database_sources' or some such. )

       'browser/renderer_host/safe_browsing_resource_throttle_factory.cc',
       'browser/renderer_host/safe_browsing_resource_throttle_factory.h',
+      'browser/renderer_host/safe_browsing_resource_throttle.h',
+      'browser/safe_browsing/database_manager.h',
+      'browser/safe_browsing/malware_details_cache.cc',
+      'browser/safe_browsing/malware_details_cache.h',

[mattm, 2015/05/08 02:17:26]

sorting got switched around here?

[Nathan Parker, 2015/05/12 01:04:01]

Was missing "LC_COLLATE=C" on the sort. Fixed.

       'browser/safe_browsing/malware_details.cc',
       'browser/safe_browsing/malware_details.h',
-      'browser/safe_browsing/malware_details_cache.cc',
-      'browser/safe_browsing/malware_details_cache.h',
       'browser/safe_browsing/malware_details_history.cc',
       'browser/safe_browsing/malware_details_history.h',
       'browser/safe_browsing/ping_manager.cc',
       'browser/safe_browsing/ping_manager.h',
       'browser/safe_browsing/protocol_manager_helper.cc',
       'browser/safe_browsing/protocol_manager_helper.h',
       'browser/safe_browsing/safe_browsing_blocking_page.cc',
       'browser/safe_browsing/safe_browsing_blocking_page.h',
+      'browser/safe_browsing/safe_browsing_database.cc',
+      'browser/safe_browsing/safe_browsing_database.h',

[mattm, 2015/05/08 02:17:26]

not needed for 2 or 3, right?

[Nathan Parker, 2015/05/12 01:04:01]

Done.

       'browser/safe_browsing/safe_browsing_service.cc',
       'browser/safe_browsing/safe_browsing_service.h',
       'browser/safe_browsing/safe_browsing_util.cc',
       'browser/safe_browsing/safe_browsing_util.h',
       'browser/safe_browsing/ui_manager.cc',
       'browser/safe_browsing/ui_manager.h',
     ],
-    # Files in addition to the "basic" ones to use for full safe browsing.
+    # "Safe Browsing Extended Mobile" files in addition to the "basic" ones
+    # to use for mobile_extended mode.
+    'chrome_browser_safe_browsing_mobile_extended_sources': [
+      'browser/safe_browsing/android_safe_browsing_api_handler.cc',
+      'browser/safe_browsing/android_safe_browsing_api_handler.h',
+      'browser/safe_browsing/remote_database_manager.cc',
+      'browser/safe_browsing/remote_database_manager.h',
+    ],
+    # "Safe Browsing Full" files in addition to the "basic" ones to use for
+    # full safe browsing.
     'chrome_browser_safe_browsing_full_sources': [
       'browser/download/download_completion_blocker.cc',
       'browser/download/download_completion_blocker.h',
-      'browser/renderer_host/safe_browsing_resource_throttle.cc',
-      'browser/renderer_host/safe_browsing_resource_throttle.h',
       'browser/safe_browsing/browser_feature_extractor.cc',
       'browser/safe_browsing/browser_feature_extractor.h',
       'browser/safe_browsing/browser_features.cc',
       'browser/safe_browsing/browser_features.h',
       'browser/safe_browsing/chunk_range.cc',
       'browser/safe_browsing/chunk_range.h',
       'browser/safe_browsing/client_side_detection_host.cc',
       'browser/safe_browsing/client_side_detection_host.h',
       'browser/safe_browsing/client_side_detection_service.cc',
       'browser/safe_browsing/client_side_detection_service.h',
-      'browser/safe_browsing/database_manager.cc',
-      'browser/safe_browsing/database_manager.h',
       'browser/safe_browsing/download_feedback.cc',
       'browser/safe_browsing/download_feedback.h',
       'browser/safe_browsing/download_feedback_service.cc',
       'browser/safe_browsing/download_feedback_service.h',
       'browser/safe_browsing/download_protection_service.cc',
       'browser/safe_browsing/download_protection_service.h',
       'browser/safe_browsing/incident_reporting/binary_integrity_analyzer.cc',
       'browser/safe_browsing/incident_reporting/binary_integrity_analyzer.h',
       'browser/safe_browsing/incident_reporting/binary_integrity_analyzer_win.cc',
       'browser/safe_browsing/incident_reporting/binary_integrity_incident.cc',
@@ skipping 34 matching lines @@
       'browser/safe_browsing/incident_reporting/resource_request_detector.cc',
       'browser/safe_browsing/incident_reporting/resource_request_detector.h',
       'browser/safe_browsing/incident_reporting/resource_request_incident.cc',
       'browser/safe_browsing/incident_reporting/resource_request_incident.h',
       'browser/safe_browsing/incident_reporting/tracked_preference_incident.cc',
       'browser/safe_browsing/incident_reporting/tracked_preference_incident.h',
       'browser/safe_browsing/incident_reporting/variations_seed_signature_analyzer.cc',
       'browser/safe_browsing/incident_reporting/variations_seed_signature_analyzer.h',
       'browser/safe_browsing/incident_reporting/variations_seed_signature_incident.cc',
       'browser/safe_browsing/incident_reporting/variations_seed_signature_incident.h',
+      'browser/safe_browsing/local_database_manager.cc',
+      'browser/safe_browsing/local_database_manager.h',
       'browser/safe_browsing/path_sanitizer.cc',
       'browser/safe_browsing/path_sanitizer.h',
       'browser/safe_browsing/prefix_set.cc',
       'browser/safe_browsing/prefix_set.h',
       'browser/safe_browsing/protocol_manager.cc',
       'browser/safe_browsing/protocol_manager.h',
       'browser/safe_browsing/protocol_parser.cc',
       'browser/safe_browsing/protocol_parser.h',
-      'browser/safe_browsing/safe_browsing_database.cc',
-      'browser/safe_browsing/safe_browsing_database.h',
       'browser/safe_browsing/safe_browsing_store.cc',
       'browser/safe_browsing/safe_browsing_store.h',
       'browser/safe_browsing/safe_browsing_store_file.cc',
       'browser/safe_browsing/safe_browsing_store_file.h',
       'browser/safe_browsing/sandboxed_zip_analyzer.cc',
       'browser/safe_browsing/sandboxed_zip_analyzer.h',
       'browser/safe_browsing/two_phase_uploader.cc',
       'browser/safe_browsing/two_phase_uploader.h',
     ],
     'chrome_browser_search_engines_sources': [
@@ skipping 874 matching lines @@
             'safe_browsing_metadata_proto',
             'safe_browsing_report_proto',
           ],
           'conditions': [
             ['safe_browsing == 1', {
               'sources': [ '<@(chrome_browser_safe_browsing_full_sources)' ],
               'dependencies': [
                 'safe_browsing_proto',
               ],
             }],
+            ['safe_browsing == 3', {
+              'sources': [ '<@(chrome_browser_safe_browsing_mobile_extended_sources)' ],
+              'dependencies': [
+                'safe_browsing_proto',
+              ],
+            }],
           ],
         }],
         ['use_udev == 1', {
           'dependencies': [
             '../device/udev_linux/udev.gyp:udev_linux',
           ],
         }],
         ['OS=="linux"', {
           'dependencies': [
             '../device/media_transfer_protocol/media_transfer_protocol.gyp:mtp_file_entry_proto',
@@ skipping 490 matching lines @@
           'variables': {
             'proto_in_dir': 'browser/sync_file_system/drive_backend',
             'proto_out_dir': 'chrome/browser/sync_file_system/drive_backend',
           },
           'includes': [ '../build/protoc.gypi' ]
         },
       ],
     },],
   ],
 }