The Reading List will be displayed as a content suggestions. It needs a provider to be interfaced with the Content Suggestions Service.
As the Reading List are content added by the user it should surface on top.
Note that we have a personal section ranker, which will (probably, soon) mostly replace the fixed ordering.
Thanks for the information.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c9bff741d84b56d8f5963a398958fe85cc110fae commit c9bff741d84b56d8f5963a398958fe85cc110fae Author: gambard <gambard@chromium.org> Date: Tue Mar 28 07:58:20 2017 Create ReadingListSuggestionsProvider This CL creates the ReadingListSuggestionsProvider. BUG= 702241 Review-Url: https://codereview.chromium.org/2755113002 Cr-Commit-Position: refs/heads/master@{#460039} [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/BUILD.gn [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/DEPS [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/category.h [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/category_rankers/constant_category_ranker.cc [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/content_suggestions_metrics.cc [add] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/reading_list/reading_list_suggestions_provider.cc [add] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/reading_list/reading_list_suggestions_provider.h [add] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/components/ntp_snippets_strings.grdp [modify] https://crrev.com/c9bff741d84b56d8f5963a398958fe85cc110fae/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/250e9409aec179c1a8427adf454309e1d2f6482f commit 250e9409aec179c1a8427adf454309e1d2f6482f Author: gambard <gambard@chromium.org> Date: Thu Mar 30 08:35:29 2017 Add logic for fetching the Reading List entries This CL adds the logic in the ReadingListProvider, allowing it to fetch the entries to be displayed. BUG= 702241 Review-Url: https://codereview.chromium.org/2770893003 Cr-Commit-Position: refs/heads/master@{#460697} [modify] https://crrev.com/250e9409aec179c1a8427adf454309e1d2f6482f/components/ntp_snippets/BUILD.gn [modify] https://crrev.com/250e9409aec179c1a8427adf454309e1d2f6482f/components/ntp_snippets/DEPS [modify] https://crrev.com/250e9409aec179c1a8427adf454309e1d2f6482f/components/ntp_snippets/reading_list/reading_list_suggestions_provider.cc [modify] https://crrev.com/250e9409aec179c1a8427adf454309e1d2f6482f/components/ntp_snippets/reading_list/reading_list_suggestions_provider.h [modify] https://crrev.com/250e9409aec179c1a8427adf454309e1d2f6482f/components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438 commit 38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438 Author: gambard <gambard@chromium.org> Date: Thu Mar 30 10:36:25 2017 Add extra information for ReadingList ContentSuggestion The Reading List suggestions need more information about the state of the distillation of the Reading List entry. BUG= 702241 Review-Url: https://codereview.chromium.org/2780793002 Cr-Commit-Position: refs/heads/master@{#460719} [modify] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/BUILD.gn [modify] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/content_suggestion.cc [modify] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/content_suggestion.h [add] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/reading_list/reading_list_distillation_state_util.cc [add] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/reading_list/reading_list_distillation_state_util.h [modify] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/reading_list/reading_list_suggestions_provider.cc [modify] https://crrev.com/38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438/components/ntp_snippets/reading_list/reading_list_suggestions_provider.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/24939285bd24532cc0c2ffca609c5e80073cd9f4 commit 24939285bd24532cc0c2ffca609c5e80073cd9f4 Author: gambard <gambard@chromium.org> Date: Thu Mar 30 12:12:11 2017 Add ContentSuggestion for ReadingList This CL allows the ContentSuggestionsUpdater to display the ReadingListContentSuggestions. BUG= 702241 , 706296 Review-Url: https://codereview.chromium.org/2755383002 Cr-Commit-Position: refs/heads/master@{#460731} [modify] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm [modify] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/cells/BUILD.gn [add] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/cells/content_suggestions_reading_list_item.h [add] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/cells/content_suggestions_reading_list_item.mm [modify] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/content_suggestion.h [modify] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.mm [modify] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/content_suggestions_view_controller.mm [modify] https://crrev.com/24939285bd24532cc0c2ffca609c5e80073cd9f4/ios/chrome/browser/ui/content_suggestions/identifier/content_suggestions_section_information.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/42692e1c75507ae5a4f9f55c357c5fda2b0a89ab commit 42692e1c75507ae5a4f9f55c357c5fda2b0a89ab Author: gambard <gambard@chromium.org> Date: Fri Mar 31 09:15:23 2017 Register the ReadingListProvider This CL register the ReadingListProvider in the ContentSuggestionsService, allowing the ReadingListEntries to be displayed. BUG= 702241 Review-Url: https://codereview.chromium.org/2790543002 Cr-Commit-Position: refs/heads/master@{#461085} [modify] https://crrev.com/42692e1c75507ae5a4f9f55c357c5fda2b0a89ab/ios/chrome/browser/ntp_snippets/BUILD.gn [modify] https://crrev.com/42692e1c75507ae5a4f9f55c357c5fda2b0a89ab/ios/chrome/browser/ntp_snippets/ios_chrome_content_suggestions_service_factory.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fea421c12c5499a4f6b04291a97705662cf41d1c commit fea421c12c5499a4f6b04291a97705662cf41d1c Author: gambard <gambard@chromium.org> Date: Wed Apr 05 08:57:12 2017 Update provider on Reading List model update When the Reading List model is updated, the provider notifies the service. BUG= 702241 Review-Url: https://codereview.chromium.org/2794853004 Cr-Commit-Position: refs/heads/master@{#462013} [modify] https://crrev.com/fea421c12c5499a4f6b04291a97705662cf41d1c/components/ntp_snippets/reading_list/reading_list_suggestions_provider.cc [modify] https://crrev.com/fea421c12c5499a4f6b04291a97705662cf41d1c/components/ntp_snippets/reading_list/reading_list_suggestions_provider.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fa80dd944e1937f0ab144bee0ac385d537bbd74a commit fa80dd944e1937f0ab144bee0ac385d537bbd74a Author: gambard <gambard@chromium.org> Date: Wed Apr 05 18:35:32 2017 Open Reading List on tap on More Forward the More action from the Reading List section to the BVC to let it open the Reading List view. BUG= 702241 Review-Url: https://codereview.chromium.org/2798883002 Cr-Commit-Position: refs/heads/master@{#462153} [modify] https://crrev.com/fa80dd944e1937f0ab144bee0ac385d537bbd74a/ios/chrome/browser/content_suggestions/BUILD.gn [modify] https://crrev.com/fa80dd944e1937f0ab144bee0ac385d537bbd74a/ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm [modify] https://crrev.com/fa80dd944e1937f0ab144bee0ac385d537bbd74a/ios/chrome/browser/content_suggestions/content_suggestions_mediator.h [modify] https://crrev.com/fa80dd944e1937f0ab144bee0ac385d537bbd74a/ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/892e302345218f7ed2ecfd9295a6011df806026e commit 892e302345218f7ed2ecfd9295a6011df806026e Author: gambard <gambard@chromium.org> Date: Thu Apr 06 10:39:07 2017 Display offline availability on ContentSuggestions This CL displays the offline availability of the ReadingList items in ContentSuggestions. BUG= 706296 , 702241 Review-Url: https://codereview.chromium.org/2796273002 Cr-Commit-Position: refs/heads/master@{#462416} [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/content_suggestions/BUILD.gn [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/content_suggestions/content_suggestions_mediator.mm [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/content_suggestions/BUILD.gn [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/content_suggestions/content_suggestion.h [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/content_suggestions/content_suggestion.mm [add] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/content_suggestions/content_suggestion_extra_builder.h [add] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/content_suggestions/content_suggestion_extra_builder.mm [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/content_suggestions/content_suggestions_collection_updater.mm [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/reading_list/reading_list_collection_view_controller.mm [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/reading_list/reading_list_utils.h [modify] https://crrev.com/892e302345218f7ed2ecfd9295a6011df806026e/ios/chrome/browser/ui/reading_list/reading_list_utils.mm
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/15343e304623319dfd5612ac3d22930977a7e20c commit 15343e304623319dfd5612ac3d22930977a7e20c Author: gambard <gambard@chromium.org> Date: Fri Apr 07 11:59:12 2017 ReadingListProvider returns empty image The Reading List provider does not have an image to provide to the UI. The FetchImage method runs the callback with an empty image. BUG= 702241 Review-Url: https://codereview.chromium.org/2802973004 Cr-Commit-Position: refs/heads/master@{#462840} [modify] https://crrev.com/15343e304623319dfd5612ac3d22930977a7e20c/components/ntp_snippets/reading_list/reading_list_suggestions_provider.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1f0337d7e2d5fc26c25f264e83a726d56ba4ee0d commit 1f0337d7e2d5fc26c25f264e83a726d56ba4ee0d Author: gambard <gambard@chromium.org> Date: Wed Apr 12 11:00:41 2017 ReadingListProvider handles dismissal Implements the logic of the dismissal of Reading List entries in their provider. BUG= 707730 , 702241 Review-Url: https://codereview.chromium.org/2815623002 Cr-Commit-Position: refs/heads/master@{#463989} [modify] https://crrev.com/1f0337d7e2d5fc26c25f264e83a726d56ba4ee0d/components/ntp_snippets/reading_list/reading_list_suggestions_provider.cc [modify] https://crrev.com/1f0337d7e2d5fc26c25f264e83a726d56ba4ee0d/components/ntp_snippets/reading_list/reading_list_suggestions_provider.h [modify] https://crrev.com/1f0337d7e2d5fc26c25f264e83a726d56ba4ee0d/components/ntp_snippets/reading_list/reading_list_suggestions_provider_unittest.cc
Comment 1 by gambard@chromium.org
, Mar 17 2017