Skip to content

Commit

Permalink
[Cleanup] Remove "TODO:rename to kRemoteSuggestionsFeature"
Browse files Browse the repository at this point in the history
    The CL removes the comment in file "components/ntp_snippets/features.h".

    There are related features with names based on the ArticleSuggestions name.
    So the CL removes the TODO and leaves the whole code intact.

BUG=
R=jkrcal@chromium.org

Signed-off-by: fr.fang <fr.fang@samsung.com>
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ie6cd5b9409b1ae6e5778e8b6c467d061cf835afe
Reviewed-on: https://chromium-review.googlesource.com/989383
Commit-Queue: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552431}
  • Loading branch information
fr.fang authored and Commit Bot committed Apr 20, 2018
1 parent 800caa6 commit fbc0b8c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 1 addition & 0 deletions AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,7 @@ Evgeniy Dushistov <dushistov@gmail.com>
Evgeny Agafonchikov <evgeny.agafonchikov@akvelon.com>
Fabien Tassin <fta@sofaraway.org>
Felix H. Dahlke <fhd@ubercode.de>
Fengrong Fang <fr.fang@samsung.com>
Fernando Jiménez Moreno <ferjmoreno@gmail.com>
Finbar Crago <finbar.crago@gmail.com>
François Beaufort <beaufort.francois@gmail.com>
Expand Down
1 change: 0 additions & 1 deletion components/ntp_snippets/features.h
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ extern const base::Feature kForeignSessionsSuggestionsFeature;
// Feature to allow show/hide article suggestions by clicking the header.
extern const base::Feature kArticleSuggestionsExpandableHeader;

// TODO(jkrcal): Rename to kRemoteSuggestionsFeature.
extern const base::Feature kArticleSuggestionsFeature;

// Feature to allow UI as specified here: https://crbug.com/660837.
Expand Down

0 comments on commit fbc0b8c

Please sign in to comment.