Otherwise it is painful to override its parameters with a command line, because it is not clear whether the ranker does not work or the command line.
Screenshots
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/147728249280f6259c9143a91d878408b4c8e5a2 commit 147728249280f6259c9143a91d878408b4c8e5a2 Author: vitaliii <vitaliii@chromium.org> Date: Thu May 18 16:52:59 2017 [NTP::SectionOrder] Add ranker section to snippets internals. Previously, it was cumbersome to verify whether ranker works (due to implicity of its effects). This CL adds some ranker debug output to snippets internals. The screenshots are in the bug. BUG= 723622 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2889153003 Cr-Commit-Position: refs/heads/master@{#472847} [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/chrome/browser/resources/snippets_internals.html [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/chrome/browser/resources/snippets_internals.js [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/chrome/browser/ui/webui/snippets_internals_message_handler.cc [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/chrome/browser/ui/webui/snippets_internals_message_handler.h [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/category_ranker.h [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/click_based_category_ranker.cc [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/click_based_category_ranker.h [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/constant_category_ranker.cc [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/constant_category_ranker.h [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/fake_category_ranker.cc [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/fake_category_ranker.h [modify] https://crrev.com/147728249280f6259c9143a91d878408b4c8e5a2/components/ntp_snippets/category_rankers/mock_category_ranker.h
Issue 675919 has been merged into this issue.
Comment 1 by vitaliii@chromium.org
, May 18 2017Status: Started (was: Assigned)