RefCounted types should not have public destructors

content/browser/renderer_host/resource_dispatcher_host_impl.cc

 // Copyright (c) 2012 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.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #include "content/browser/renderer_host/resource_dispatcher_host_impl.h"
 
 #include <set>
 #include <vector>
@@ skipping 177 matching lines @@
         return false;
       }
     }
   }
 
   return true;
 }
 
 void PopulateResourceResponse(net::URLRequest* request,
                               ResourceResponse* response) {
-  response->status = request->status();
-  response->request_time = request->request_time();
-  response->response_time = request->response_time();
-  response->headers = request->response_headers();
-  request->GetCharset(&response->charset);
-  response->content_length = request->GetExpectedContentSize();
-  request->GetMimeType(&response->mime_type);
+  response->data.status = request->status();
+  response->data.request_time = request->request_time();
+  response->data.response_time = request->response_time();
+  response->data.headers = request->response_headers();
+  request->GetCharset(&response->data.charset);
+  response->data.content_length = request->GetExpectedContentSize();
+  request->GetMimeType(&response->data.mime_type);
   net::HttpResponseInfo response_info = request->response_info();
-  response->was_fetched_via_spdy = response_info.was_fetched_via_spdy;
-  response->was_npn_negotiated = response_info.was_npn_negotiated;
-  response->npn_negotiated_protocol = response_info.npn_negotiated_protocol;
-  response->was_fetched_via_proxy = request->was_fetched_via_proxy();
-  response->socket_address = request->GetSocketAddress();
+  response->data.was_fetched_via_spdy = response_info.was_fetched_via_spdy;
+  response->data.was_npn_negotiated = response_info.was_npn_negotiated;
+  response->data.npn_negotiated_protocol =
+      response_info.npn_negotiated_protocol;
+  response->data.was_fetched_via_proxy = request->was_fetched_via_proxy();
+  response->data.socket_address = request->GetSocketAddress();
   appcache::AppCacheInterceptor::GetExtraResponseInfo(
       request,
-      &response->appcache_id,
-      &response->appcache_manifest_url);
+      &response->data.appcache_id,
+      &response->data.appcache_manifest_url);
 }
 
 void RemoveDownloadFileFromChildSecurityPolicy(int child_id,
                                                const FilePath& path) {
   ChildProcessSecurityPolicyImpl::GetInstance()->RevokeAllPermissionsForFile(
       child_id, path);
 }
 
 #if defined(OS_WIN)
 #pragma warning(disable: 4748)
@@ skipping 1352 matching lines @@
     net::URLRequest* request) {
   ResourceRequestInfoImpl* info = ResourceRequestInfoImpl::ForRequest(request);
 
   scoped_refptr<ResourceResponse> response(new ResourceResponse);
   PopulateResourceResponse(request, response);
 
   if (request->ssl_info().cert) {
     int cert_id =
         CertStore::GetInstance()->StoreCert(request->ssl_info().cert,
                                             info->GetChildID());
-    response->security_info = SerializeSecurityInfo(
+    response->data.security_info = SerializeSecurityInfo(
         cert_id, request->ssl_info().cert_status,
         request->ssl_info().security_bits,
         request->ssl_info().connection_status);
   } else {
     // We should not have any SSL state.
     DCHECK(!request->ssl_info().cert_status &&
            request->ssl_info().security_bits == -1 &&
            !request->ssl_info().connection_status);
   }
 
@@ skipping 736 matching lines @@
   return allow_cross_origin_auth_prompt_;
 }
 
 bool ResourceDispatcherHostImpl::IsTransferredNavigation(
     const GlobalRequestID& transferred_request_id) const {
   return transferred_navigations_.find(transferred_request_id) !=
       transferred_navigations_.end();
 }
 
 }  // namespace content