Clean up TestFeaturesNativeHandler to use the ExtensionsClient's JSON feature sources.

extensions/common/extensions_client.h

Index: extensions/common/extensions_client.h
diff --git a/extensions/common/extensions_client.h b/extensions/common/extensions_client.h
index 6b9c5349b01271fca1ce4323fd36ea2b9773e434..d8f58ef3b0d03d79aafc9208a41bab008a4549a4 100644
--- a/extensions/common/extensions_client.h
+++ b/extensions/common/extensions_client.h
@@ -19,6 +19,7 @@ namespace extensions {
 class APIPermissionSet;
 class Extension;
 class FeatureProvider;
+class JSONFeatureProviderSource;
 class ManifestPermissionSet;
 class PermissionMessage;
 class PermissionMessageProvider;
@@ -48,6 +49,12 @@ class ExtensionsClient {
   virtual scoped_ptr<FeatureProvider> CreateFeatureProvider(
       const std::string& name) const = 0;
 
+  // Create a JSONFeatureProviderSource for a specific feature type,
+  // e.g. "permission". (There are currently no other sources of features data.)

[James Cook, 2014/05/20 18:15:47]

Doesn't this also work for "api" and "manifest"?  At least the
ShellExtensionsClient works for those.

[Yoyo Zhou, 2014/05/21 01:05:47]

This wording is confusing. I'll fix it.

+  // This is used primarily in CreateFeatureProvider, above.
+  virtual scoped_ptr<JSONFeatureProviderSource> CreateFeatureProviderSource(
+      const std::string& name) const = 0;
+
   // Takes the list of all hosts and filters out those with special
   // permission strings. Adds the regular hosts to |new_hosts|,
   // and adds the special permission messages to |messages|.