Further reduction of feature_compiler.py generated code

extensions/common/features/base_feature_provider.cc

Index: extensions/common/features/base_feature_provider.cc
diff --git a/extensions/common/features/base_feature_provider.cc b/extensions/common/features/base_feature_provider.cc
index adc463cc04e9b6a54542995d055c5aee1ec860b6..26a4e740b9e1c660348ae733bacb3d75006b58eb 100644
--- a/extensions/common/features/base_feature_provider.cc
+++ b/extensions/common/features/base_feature_provider.cc
@@ -66,4 +66,8 @@ void BaseFeatureProvider::AddFeature(base::StringPiece name,
   features_[name.as_string()] = std::move(feature);
 }
 
+void BaseFeatureProvider::AddFeature(base::StringPiece name, Feature* feature) {
+  features_[name.as_string()] = std::unique_ptr<Feature>(feature);

[Devlin, 2016/07/29 20:13:49]

For the record, this is what I was talking about with FeatureProvider - we still
have features_ has an STL container of unique_ptrs, but just pass it in by raw
ptr and then add it to the container as a unique_ptr.  With the complex feature,
we already have the container, so maybe we could avoid the wrap/unwrap just
through the generated code.  See other comment.

+}
+
 }  // namespace extensions