initial add of SQL based storage

components/offline_pages/BUILD.gn

 # Copyright 2015 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.
 
 if (is_android) {
   import("//build/config/android/rules.gni")
 }
 
 # GYP: //components/offline_pages.gypi:offline_pages
 static_library("offline_pages") {
   sources = [
     "offline_page_archiver.h",
     "offline_page_bookmark_bridge.cc",
     "offline_page_bookmark_bridge.h",
     "offline_page_item.cc",
     "offline_page_item.h",
     "offline_page_metadata_store.cc",
     "offline_page_metadata_store.h",
     "offline_page_metadata_store_impl.cc",
     "offline_page_metadata_store_impl.h",
+    "offline_page_metadata_store_sql.cc",
+    "offline_page_metadata_store_sql.h",
     "offline_page_model.cc",
     "offline_page_model.h",
   ]
 
   deps = [
     ":switches",
     "//base",
     "//components/bookmarks/browser",
     "//components/keyed_service/core",
     "//components/leveldb_proto",
     "//components/offline_pages/proto:offline_pages_proto",
     "//net",
+    "//sql",
     "//third_party/leveldatabase",
     "//url",
   ]
 }
 
 static_library("test_support") {
   testonly = true
   sources = [
     "offline_page_test_archiver.cc",
     "offline_page_test_archiver.h",
@@ skipping 34 matching lines @@
   deps = [
     ":offline_pages",
     ":switches",
     ":test_support",
     "//base",
     "//base/test:test_support",
     "//components/bookmarks/browser",
     "//components/bookmarks/test",
     "//components/leveldb_proto",
     "//components/offline_pages/proto:offline_pages_proto",
+    "//sql:sql",

[Scott Hess - ex-Googler, 2016/04/25 19:57:55]

Why is this //sql:sql while the other was //sql?

[bburns, 2016/04/26 23:27:02]

good question, changed the other to sql:sql (they're equivalent, but the
consistency is good, I suppose)

[Scott Hess - ex-Googler, 2016/04/27 15:21:00]

In that case ... why //sql:sql but not //url:url?

[I still can't figure out if this little voice in my head is a defect or not!]

[bburns, 2016/04/27 19:59:21]

if you do "//foo" it implicitly means "//foo:foo"

so I guess it's a style thing, I'm not sure there is a ton of consistency around
this, but I don't super care.

[Scott Hess - ex-Googler, 2016/04/27 21:16:45]

I'm fine with a "don't care", mostly I was pointing it out because the case
you're adding is the only case in this file where it's //x:x.

     "//testing/gtest",
     "//url",
   ]
 }
 
 if (is_android) {
   java_cpp_enum("offline_page_feature_enums_java") {
     sources = [
       "offline_page_feature.h",
     ]
   }
 
   java_cpp_enum("offline_page_model_enums_java") {
     sources = [
       "offline_page_model.h",
     ]
   }
 }