[Offline pages] Adding persistent request queue based on SQLite

components/components_tests.gyp

Index: components/components_tests.gyp
diff --git a/components/components_tests.gyp b/components/components_tests.gyp
index c36f13c37142df2730ce2024ef9f4924f4a4dd44..6ffb2ba92050faf4f9ea23cbe369bd8462dd8d1f 100644
--- a/components/components_tests.gyp
+++ b/components/components_tests.gyp
@@ -82,8 +82,8 @@
       'browser_sync/browser/profile_sync_service_autofill_unittest.cc',
       'browser_sync/browser/profile_sync_service_bookmark_unittest.cc',
       'browser_sync/browser/profile_sync_service_startup_unittest.cc',
-      'browser_sync/browser/profile_sync_service_unittest.cc',
       'browser_sync/browser/profile_sync_service_typed_url_unittest.cc',
+      'browser_sync/browser/profile_sync_service_unittest.cc',
     ],
     'browser_watcher_unittest_sources': [
       'browser_watcher/endsession_watcher_window_win_unittest.cc',
@@ -371,11 +371,11 @@
       'metrics/call_stack_profile_metrics_provider_unittest.cc',
       'metrics/cloned_install_detector_unittest.cc',
       'metrics/daily_event_unittest.cc',
+      'metrics/data_use_tracker_unittest.cc',
       'metrics/drive_metrics_provider_unittest.cc',
       'metrics/file_metrics_provider_unittest.cc',
       'metrics/histogram_encoder_unittest.cc',
       'metrics/machine_id_provider_win_unittest.cc',
-      'metrics/data_use_tracker_unittest.cc',
       'metrics/metrics_log_manager_unittest.cc',
       'metrics/metrics_log_unittest.cc',
       'metrics/metrics_reporting_scheduler_unittest.cc',
@@ -392,8 +392,8 @@
       'metrics/leak_detector/call_stack_manager_unittest.cc',
       'metrics/leak_detector/call_stack_table_unittest.cc',
       'metrics/leak_detector/leak_analyzer_unittest.cc',
-      'metrics/leak_detector/leak_detector_unittest.cc',
       'metrics/leak_detector/leak_detector_impl_unittest.cc',
+      'metrics/leak_detector/leak_detector_unittest.cc',
       'metrics/leak_detector/ranked_set_unittest.cc',
     ],
     'mime_util_unittest_sources': [
@@ -432,7 +432,7 @@
     'offline_pages_background_unittest_sources': [
       'offline_pages/background/request_coordinator_unittest.cc',
       'offline_pages/background/request_picker_unittest.cc',
-      'offline_pages/background/request_queue_in_memory_store_unittest.cc',
+      'offline_pages/background/request_queue_store_unittest.cc',

[Pete Williamson, 2016/06/14 21:54:45]

Should we keep the in memory store unittest?  The memory store sure is handy for
other unit tests. 

[fgorski, 2016/06/14 22:38:05]

we are, we are converting the test to a typed test to make sure both stores work
the same way.

       'offline_pages/background/request_queue_unittest.cc',
       'offline_pages/background/save_page_request_unittest.cc',
     ],
@@ -507,8 +507,8 @@
       'password_manager/core/browser/export/password_exporter_unittest.cc',
       'password_manager/core/browser/facet_manager_unittest.cc',
       'password_manager/core/browser/import/csv_reader_unittest.cc',
-      'password_manager/core/browser/import/password_importer_unittest.cc',
       'password_manager/core/browser/import/password_csv_reader_unittest.cc',
+      'password_manager/core/browser/import/password_importer_unittest.cc',
       'password_manager/core/browser/log_manager_unittest.cc',
       'password_manager/core/browser/log_router_unittest.cc',
       'password_manager/core/browser/login_database_ios_unittest.cc',
@@ -808,8 +808,8 @@
       'sync_driver/device_info_util_unittest.cc',
       'sync_driver/frontend_data_type_controller_unittest.cc',
       'sync_driver/generic_change_processor_unittest.cc',
-      'sync_driver/glue/sync_backend_host_impl_unittest.cc',
       'sync_driver/glue/browser_thread_model_worker_unittest.cc',
+      'sync_driver/glue/sync_backend_host_impl_unittest.cc',
       'sync_driver/glue/sync_backend_registrar_unittest.cc',
       'sync_driver/glue/ui_model_worker_unittest.cc',
       'sync_driver/local_device_info_provider_unittest.cc',
@@ -836,8 +836,8 @@
       'sync_sessions/revisit/sessions_page_revisit_observer_unittest.cc',
       'sync_sessions/revisit/typed_url_page_revisit_task_unittest.cc',
       'sync_sessions/session_data_type_controller_unittest.cc',
-      'sync_sessions/synced_session_tracker_unittest.cc',
       'sync_sessions/sync_sessions_metrics_unittest.cc',
+      'sync_sessions/synced_session_tracker_unittest.cc',
       'sync_sessions/tab_node_pool_unittest.cc',
     ],
     'syncable_prefs_unittest_sources': [
@@ -850,8 +850,8 @@
       'tracing/common/process_metrics_memory_dump_provider_unittest.cc',
     ],
     'translate_unittest_sources': [
-      'translate/core/browser/mock_translate_driver.cc',
       'translate/core/browser/language_state_unittest.cc',
+      'translate/core/browser/mock_translate_driver.cc',
       'translate/core/browser/translate_browser_metrics_unittest.cc',
       'translate/core/browser/translate_language_list_unittest.cc',
       'translate/core/browser/translate_manager_unittest.cc',