Currently ClickBasedCategoryRanker does not know anything about section dismissal, however, this may be a good signal that the category must be moved down.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3d9423e34805ed9a6046d44e6ce144a39b16f603 commit 3d9423e34805ed9a6046d44e6ce144a39b16f603 Author: vitaliii <vitaliii@chromium.org> Date: Tue Jan 03 17:08:13 2017 [NTP::SectionOrder] Make ClickBasedRanker move dismissed sections down. Move dissmised sections 2 positions down in ClickBasedRanker. Dismissing a section is a strong negative signal, thus, category rankers can react to it. BUG= 677781 Review-Url: https://codereview.chromium.org/2610553002 Cr-Commit-Position: refs/heads/master@{#441140} [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/category_ranker.h [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/click_based_category_ranker.cc [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/click_based_category_ranker.h [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/click_based_category_ranker_unittest.cc [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/constant_category_ranker.cc [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/constant_category_ranker.h [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/fake_category_ranker.cc [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/fake_category_ranker.h [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/category_rankers/mock_category_ranker.h [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/content_suggestions_service.cc [modify] https://crrev.com/3d9423e34805ed9a6046d44e6ce144a39b16f603/components/ntp_snippets/content_suggestions_service_unittest.cc
In the end, it is "1 position down" and not "2 positions down".
Comment 1 by vitaliii@chromium.org
, Jan 2 2017Blocking: 646841