Second patch in making destructors of refcounted objects private.

net/url_request/url_request_job.h

 // Copyright (c) 2006-2009 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 NET_URL_REQUEST_URL_REQUEST_JOB_H_
 #define NET_URL_REQUEST_URL_REQUEST_JOB_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 23 matching lines @@
                       public FilterContext {
  public:
   // When histogramming results related to SDCH and/or an SDCH latency test, the
   // number of packets for which we need to record arrival times so as to
   // calculate interpacket latencies.  We currently are only looking at the
   // first few packets, as we're monitoring the impact of the initial TCP
   // congestion window on stalling of transmissions.
   static const size_t kSdchPacketHistogramCount = 5;
 
   explicit URLRequestJob(URLRequest* request);
-  virtual ~URLRequestJob();
 
   // Returns the request that owns this job. THIS POINTER MAY BE NULL if the
   // request was destroyed.
   URLRequest* request() const {
     return request_;
   }
 
   // Sets the upload data, most requests have no upload data, so this is a NOP.
   // Job types supporting upload data will override this.
   virtual void SetUpload(net::UploadData* upload) { }
@@ skipping 156 matching lines @@
   virtual bool GetMimeType(std::string* mime_type) const { return false; }
   virtual bool GetURL(GURL* gurl) const;
   virtual base::Time GetRequestTime() const;
   virtual bool IsCachedContent() const { return false; }
   virtual int64 GetByteReadCount() const;
   virtual int GetResponseCode() const { return -1; }
   virtual int GetInputStreamBufferSize() const { return kFilterBufSize; }
   virtual void RecordPacketStats(StatisticSelector statistic) const;
 
  protected:
+  friend class base::RefCountedThreadSafe<URLRequestJob>;
+  virtual ~URLRequestJob();
+
   // Notifies the job that headers have been received.
   void NotifyHeadersComplete();
 
   // Notifies the request that the job has completed a Read operation.
   void NotifyReadComplete(int bytes_read);
 
   // Notifies the request that a start error has occurred.
   void NotifyStartError(const URLRequestStatus& status);
 
   // NotifyDone marks when we are done with a request.  It is really
@@ skipping 163 matching lines @@
   base::Time final_packet_time_;
 
   // The count of the number of packets, some of which may not have been timed.
   // We're ignoring overflow, as 1430 x 2^31 is a LOT of bytes.
   int observed_packet_count_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequestJob);
 };
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_JOB_H_