chrome/browser/android/chrome_feature_list.cc - Issue 2214683002: [New CL] Add bookmark provider for content suggestions

Side by Side Diff: chrome/browser/android/chrome_feature_list.cc

Issue 2214683002: [New CL] Add bookmark provider for content suggestions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bookmarks

Patch Set: Move the feature to components Created 4 years, 4 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/ntp_snippets/content_suggestions_service_factory.cc » ('j') | components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 // Copyright 2015 The Chromium Authors. All rights reserved.	1 // Copyright 2015 The Chromium Authors. All rights reserved.

2 // Use of this source code is governed by a BSD-style license that can be	2 // Use of this source code is governed by a BSD-style license that can be

3 // found in the LICENSE file.	3 // found in the LICENSE file.

4	4

5 #include "chrome/browser/android/chrome_feature_list.h"	5 #include "chrome/browser/android/chrome_feature_list.h"

6	6

7 #include <stddef.h>	7 #include <stddef.h>

8	8

9 #include <string>	9 #include <string>

10	10

(...skipping 18 matching lines...) Expand all Loading...
29 // this array may either refer to features defined in the header of this file or	29 // this array may either refer to features defined in the header of this file or

30 // in other locations in the code base (e.g. chrome/, components/, etc).	30 // in other locations in the code base (e.g. chrome/, components/, etc).

31 const base::Feature* kFeaturesExposedToJava[] = {	31 const base::Feature* kFeaturesExposedToJava[] = {

32 &features::kAutoplayMutedVideos,	32 &features::kAutoplayMutedVideos,

33 &features::kCredentialManagementAPI,	33 &features::kCredentialManagementAPI,

34 &features::kSimplifiedFullscreenUI,	34 &features::kSimplifiedFullscreenUI,

35 &features::kWebPayments,	35 &features::kWebPayments,

36 &kAllBookmarksFeature,	36 &kAllBookmarksFeature,

37 &kAndroidPayIntegrationV1,	37 &kAndroidPayIntegrationV1,

38 &kImportantSitesInCBD,	38 &kImportantSitesInCBD,

	39 &ntp_snippets::kBookmarkSuggestionsFeature,
	Marc Treib 2016/08/04 13:57:14 I don't think this needs to be here? Does any Java I don't think this needs to be here? Does any Java code query this feature? Philipp Keck 2016/08/04 14:43:42 Done. Show quoted text On 2016/08/04 13:57:14, Marc Treib wrote: > I don't think this needs to be here? Does any Java code query this feature? Done.
39 &ntp_snippets::kContentSuggestionsFeature,	40 &ntp_snippets::kContentSuggestionsFeature,

40 &kNTPFakeOmniboxTextFeature,	41 &kNTPFakeOmniboxTextFeature,

41 &kNTPMaterialDesign,	42 &kNTPMaterialDesign,

42 &kNTPOfflinePagesFeature,	43 &kNTPOfflinePagesFeature,

43 &kNTPOfflinePageSuggestionsFeature,	44 &kNTPOfflinePageSuggestionsFeature,

44 &kNTPToolbarFeature,	45 &kNTPToolbarFeature,

45 &kPhysicalWebFeature,	46 &kPhysicalWebFeature,

46 &kPhysicalWebIgnoreOtherClientsFeature,	47 &kPhysicalWebIgnoreOtherClientsFeature,

47 &kReadItLaterInMenu,	48 &kReadItLaterInMenu,

48 &kSystemDownloadManager,	49 &kSystemDownloadManager,

(...skipping 58 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
107 NOTREACHED();	108 NOTREACHED();

108 return false;	109 return false;

109 }	110 }

110	111

111 bool RegisterChromeFeatureListJni(JNIEnv* env) {	112 bool RegisterChromeFeatureListJni(JNIEnv* env) {

112 return RegisterNativesImpl(env);	113 return RegisterNativesImpl(env);

113 }	114 }

114	115

115 } // namespace android	116 } // namespace android

116 } // namespace chrome	117 } // namespace chrome

OLD	NEW