[Offline Pages] Feature flag to allow concurrent background loading on svelte

chrome/browser/android/chrome_feature_list.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/chrome_feature_list.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 43 matching lines @@
     &kWebApks,
     &ntp_snippets::kContentSuggestionsFeature,
     &ntp_snippets::kIncreasedVisibility,
     &ntp_snippets::kOfflineBadgeFeature,
     &ntp_snippets::kSaveToOfflineFeature,
     &ntp_snippets::kSectionDismissalFeature,
     &offline_pages::kBackgroundLoaderForDownloadsFeature,
     &offline_pages::kOfflinePagesBackgroundLoadingFeature,
     &offline_pages::kOfflinePagesCTFeature,  // See crbug.com/620421.
     &offline_pages::kOfflinePagesSharingFeature,
+    &offline_pages::kOfflinePagesSvelteConcurrentLoadingFeature,

[Ted C, 2016/11/08 17:42:13]

This array says these are the features exposed to java but I don't see this used
anywhere in the java code.  Do you actually need this here?

Are any of these offline entries accessed from java?

I'd vote for not adding things to this list until it is needed.

[dougarnett, 2016/11/08 18:04:26]

Done, thanks

     &password_manager::features::kViewPasswords,
 };
 
 }  // namespace
 
 // Alphabetical:
 const base::Feature kAndroidPayIntegrationV1{"AndroidPayIntegrationV1",
                                              base::FEATURE_ENABLED_BY_DEFAULT};
 
 const base::Feature kCCTExternalLinkHandling{"CCTExternalLinkHandling",
@@ skipping 54 matching lines @@
   NOTREACHED();
   return false;
 }
 
 bool RegisterChromeFeatureListJni(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android
 }  // namespace chrome