Enable RL by default

components/reading_list/core/reading_list.gni

Index: components/reading_list/core/reading_list.gni
diff --git a/components/reading_list/core/BUILD.gn b/components/reading_list/core/reading_list.gni
similarity index 55%
copy from components/reading_list/core/BUILD.gn
copy to components/reading_list/core/reading_list.gni
index 52dbc37326d1c354765936715725ca02e3978b5f..b015a901e5530b550621c6e503a8c263a424c56f 100644
--- a/components/reading_list/core/BUILD.gn
+++ b/components/reading_list/core/reading_list.gni
@@ -2,12 +2,8 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-source_set("core") {
-  sources = [
-    "reading_list_switches.cc",
-    "reading_list_switches.h",
-  ]
-  deps = [
-    "//base",
-  ]
+# This file declares build flags for Reading List.

[sdefresne, 2016/12/09 15:10:59]

Remove, this is obvious.

[Olivier, 2016/12/09 15:27:38]

Done.

+
+declare_args() {
+  enable_reading_list = true

[sdefresne, 2016/12/09 15:10:59]

Can you add a comment to document this variable? Maybe something like this.

  # Controls whether reading list support is active or not. Currently only
  # supported on iOS (on other platforms, the variable is ignored).
  enable_reading_list = true

The reason is that you can get a list of all variables with documentation by
using "gn args --list out/xxx". For variables defined in declare_args, the
documentation is the comment above the variable definition.

[Olivier, 2016/12/09 15:27:38]

Done.

 }