Reading List create protobuf store

ios/chrome/BUILD.gn

Index: ios/chrome/BUILD.gn
diff --git a/ios/chrome/BUILD.gn b/ios/chrome/BUILD.gn
index 14e6a75a1fefe7e216962940de445bdbf1b1139e..fa1b96df859861a8eef32e8c505cf33ad895d9b1 100644
--- a/ios/chrome/BUILD.gn
+++ b/ios/chrome/BUILD.gn
@@ -60,8 +60,8 @@ test("ios_chrome_unittests") {
     "browser/passwords/password_controller_unittest.mm",
     "browser/passwords/password_generation_agent_unittest.mm",
     "browser/reading_list/reading_list_entry_unittest.cc",
-    "browser/reading_list/reading_list_model_storage_unittest.mm",
     "browser/reading_list/reading_list_model_unittest.cc",
+    "browser/reading_list/reading_list_store_unittest.cc",
     "browser/reading_list/url_downloader_unittest.cc",
     "browser/signin/chrome_identity_service_observer_bridge_unittest.mm",
     "browser/signin/gaia_auth_fetcher_ios_unittest.mm",
@@ -99,6 +99,7 @@ test("ios_chrome_unittests") {
     "//components/bookmarks/test",
     "//components/browsing_data/core",
     "//components/favicon_base",
+    "//components/leveldb_proto:test_support",
     "//components/metrics",
     "//components/metrics:test_support",
     "//components/password_manager/core/browser",
@@ -117,6 +118,7 @@ test("ios_chrome_unittests") {
     "//ios/chrome/app/resources:packed_resources",
     "//ios/chrome/browser",
     "//ios/chrome/browser:test_support",
+    "//ios/chrome/browser/reading_list/proto",
     "//ios/chrome/common",
     "//ios/chrome/test",
     "//ios/chrome/test:run_all_unittests",
@@ -137,5 +139,7 @@ test("ios_chrome_unittests") {
     "//ui/strings:ui_strings",
   ]
 
+  include_dirs = [ "$root_gen_dir/components/sync/protocol" ]

[sdefresne, 2016/10/06 15:11:24]

You should never have to do that. I think you are missing a dependency (or the
target that generate this should use public_configs instead of configs).

I think ios/chrome/browser/reading_list/proto probably need to also define a
config with this include_dirs, and add it to its public_configs (see my comment
in the other CL).

Even then, I think it strange that you use
"$root_gen_dir/components/sync/protocol", it should be ""$root_gen_dir" and then
you should prefix your paths with "components/sync/protocol".

+
   assert_no_deps = ios_assert_no_deps
 }