Read feature param for Physical Web experiment

chrome/browser/android/chrome_feature_list.cc

Index: chrome/browser/android/chrome_feature_list.cc
diff --git a/chrome/browser/android/chrome_feature_list.cc b/chrome/browser/android/chrome_feature_list.cc
index 35a046c636627172fea258c1daa4331bdfa42376..4cf756ed74bcf482f6037dbf25225db0ca5475ad 100644
--- a/chrome/browser/android/chrome_feature_list.cc
+++ b/chrome/browser/android/chrome_feature_list.cc
@@ -7,7 +7,6 @@
 #include <string>
 
 #include "base/android/jni_string.h"
-#include "base/feature_list.h"
 #include "jni/ChromeFeatureList_jni.h"
 
 using base::android::ConvertJavaStringToUTF8;
@@ -17,10 +16,6 @@ namespace android {
 
 namespace {
 
-const base::Feature kPhysicalWebFeature {
-  "PhysicalWeb", base::FEATURE_DISABLED_BY_DEFAULT
-};
-
 // Array of features exposed through the Java ChromeFeatureList API. Entries in
 // this array may either refer to features defined in this file (above) or in
 // other locations in the code base (e.g. chrome/, components/, etc).
@@ -47,5 +42,9 @@ bool RegisterChromeFeatureListJni(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
+const base::Feature kPhysicalWebFeature {

[mmocny, 2015/12/21 15:45:55]

I don't expect its a good idea to use static initialization of a non-POD type
like this.

I'm not sure what the style guide for chromium says about it, but typically you
either wrap in a Get*() function which itself holds the static data, or create a
global plain *pointer* to the object.

[Alexei Svitkine (slow), 2015/12/21 15:50:50]

It's a POD type and this is the correct way to initialize it. (See the docs in
base/feature_list.h)

+  "PhysicalWeb", base::FEATURE_DISABLED_BY_DEFAULT
+};

[Alexei Svitkine (slow), 2015/12/21 15:48:18]

Nit: I'd put this after line 27.

[cco3, 2015/12/21 18:10:11]

Done.

+
 }  // namespace android
 }  // namespace chrome