Kill URLRequestJobTracker.

net/url_request/url_request_test_util.cc

 // Copyright (c) 2011 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.
 
 #include "net/url_request/url_request_test_util.h"
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/threading/thread.h"
@@ skipping 261 matching lines @@
                                            const GURL& new_location) {
 }
 
 void TestNetworkDelegate::OnResponseStarted(net::URLRequest* request) {
   if (request->status().status() == net::URLRequestStatus::FAILED) {
     error_count_++;
     last_os_error_ = request->status().os_error();
   }
 }
 
+void TestNetworkDelegate::OnBytesRead(const net::URLRequest& /* request */,
+                                      int /* bytes_read */) {}

[rvargas (doing something else), 2011/05/18 21:45:18]

nit: Same comment here about the argument name. And I guess that what looks bad
to me are multi-line definitions that end up with {}. See the rest of this file.

+
 void TestNetworkDelegate::OnCompleted(net::URLRequest* request) {
   if (request->status().status() == net::URLRequestStatus::FAILED) {
     error_count_++;
     last_os_error_ = request->status().os_error();
   }
 }
 
 void TestNetworkDelegate::OnURLRequestDestroyed(net::URLRequest* request) {
   destroyed_requests_++;
 }
 
 void TestNetworkDelegate::OnHttpTransactionDestroyed(uint64 request_id) {
 }
 
 net::URLRequestJob* TestNetworkDelegate::OnMaybeCreateURLRequestJob(
     net::URLRequest* request) {
   return NULL;
 }
 
 void TestNetworkDelegate::OnPACScriptError(int line_number,
                                            const string16& error) {
 }