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

extensions/test/test_extensions_client.h

 // Copyright 2014 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.
 
 #ifndef EXTENSIONS_TEST_TEST_EXTENSIONS_CLIENT_H_
 #define EXTENSIONS_TEST_TEST_EXTENSIONS_CLIENT_H_
 
 #include "base/macros.h"
 #include "extensions/common/extensions_client.h"
 
 namespace extensions {
 
 class TestExtensionsClient : public ExtensionsClient {
  public:
   TestExtensionsClient();
   virtual ~TestExtensionsClient();
 
  private:
   virtual void Initialize() OVERRIDE;
-  virtual const PermissionsProvider& GetPermissionsProvider() const OVERRIDE;

[Yoyo Zhou, 2014/05/20 01:49:34]

This was missed in a previous refactoring because extensions_unittests doesn't
compile on the trybots. =/

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

I think tfarina addressed this in https://codereview.chromium.org/294483003/ so
you guys will be in a race to land/rebase. :-)

   virtual const PermissionMessageProvider& GetPermissionMessageProvider() const
       OVERRIDE;
   virtual scoped_ptr<FeatureProvider> CreateFeatureProvider(
       const std::string& name) const OVERRIDE;
+  virtual scoped_ptr<JSONFeatureProviderSource> CreateFeatureProviderSource(
+      const std::string& name) const OVERRIDE;
   virtual void FilterHostPermissions(
       const URLPatternSet& hosts,
       URLPatternSet* new_hosts,
       std::set<PermissionMessage>* messages) const OVERRIDE;
   virtual void SetScriptingWhitelist(
       const ScriptingWhitelist& whitelist) OVERRIDE;
   virtual const ScriptingWhitelist& GetScriptingWhitelist() const OVERRIDE;
   virtual URLPatternSet GetPermittedChromeSchemeHosts(
       const Extension* extension,
       const APIPermissionSet& api_permissions) const OVERRIDE;
   virtual bool IsScriptableURL(const GURL& url,
                                std::string* error) const OVERRIDE;
   virtual bool IsAPISchemaGenerated(const std::string& name) const OVERRIDE;
   virtual base::StringPiece GetAPISchema(
       const std::string& name) const OVERRIDE;
   virtual bool ShouldSuppressFatalErrors() const OVERRIDE;
 
   // A whitelist of extensions that can script anywhere. Do not add to this
   // list (except in tests) without consulting the Extensions team first.
   // Note: Component extensions have this right implicitly and do not need to be
   // added to this list.
   ScriptingWhitelist scripting_whitelist_;
 
   DISALLOW_COPY_AND_ASSIGN(TestExtensionsClient);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_TEST_TEST_EXTENSIONS_CLIENT_H_