| Index: components/ntp_snippets/remote/remote_suggestions_fetcher_unittest.cc
|
| diff --git a/components/ntp_snippets/remote/remote_suggestions_fetcher_unittest.cc b/components/ntp_snippets/remote/remote_suggestions_fetcher_unittest.cc
|
| index 77b57ecaa02f139689d0bfe5b8616fd407252307..246372e7920b6797686c8d2310b4f0a53152f3a5 100644
|
| --- a/components/ntp_snippets/remote/remote_suggestions_fetcher_unittest.cc
|
| +++ b/components/ntp_snippets/remote/remote_suggestions_fetcher_unittest.cc
|
| @@ -399,9 +399,11 @@ class RemoteSuggestionsSignedOutFetcherTest
|
| GURL(kTestChromeContentSuggestionsSignedOutUrl)) {}
|
| };
|
|
|
| -// TODO(jkrcal): Add unit-tests for the "authentication in progress" case as it
|
| -// requires more changes (instead FakeSigninManagerBase use FakeSigninManager
|
| -// which does not exist on ChromeOS). crbug.com/688310
|
| +// TODO(jkrcal): Investigate whether the "authentication in progress" case can
|
| +// ever happen (see discussion on https://codereview.chromium.org/2582573002),
|
| +// and if so, add unit-tests for it. This will require more changes (instead of
|
| +// FakeSigninManagerBase use FakeSigninManager which does not exist on
|
| +// ChromeOS). crbug.com/688310
|
| class RemoteSuggestionsSignedInFetcherTest
|
| : public RemoteSuggestionsFetcherTestBase {
|
| public:
|
|
|