RefCounted types should not have public destructors, net/

net/base/file_stream_net_log_parameters.cc

Index: net/base/file_stream_net_log_parameters.cc
diff --git a/net/base/file_stream_net_log_parameters.cc b/net/base/file_stream_net_log_parameters.cc
index 3a8c42fefab2d9bce3d98bd49aa18271512c667b..731e73952a93f48b6483d150da423f351daea994 100644
--- a/net/base/file_stream_net_log_parameters.cc
+++ b/net/base/file_stream_net_log_parameters.cc
@@ -13,6 +13,8 @@ FileStreamErrorParameters::FileStreamErrorParameters(
         : operation_(operation), os_error_(os_error), net_error_(net_error) {
 }
 
+FileStreamErrorParameters::~FileStreamErrorParameters() {}

[eroman, 2012/04/13 18:59:34]

Please match declaration order -- i.e. put this after ToValue().

This comment applies to other places in this CL. Note that I don't expect you to
do it for all the refactorings that occur in this CL, but it is something to
keep in mind.

+
 Value* FileStreamErrorParameters::ToValue() const {
   DictionaryValue* dict = new DictionaryValue();