Don't destroy a request while it is being processed

chrome/browser/download/download_request_limiter_unittest.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/download/download_request_limiter.h"
 #include "chrome/test/testing_profile.h"
 #include "content/browser/browser_thread.h"
 #include "content/browser/renderer_host/test_render_view_host.h"
 #include "content/browser/tab_contents/navigation_controller.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 22 matching lines @@
     RenderViewHostTestHarness::TearDown();
   }
 
   virtual void ContinueDownload() {
     continue_count_++;
   }
   virtual void CancelDownload() {
     cancel_count_++;
   }
 
-  virtual int GetRequestId() {
-    return -1;
-  }
-
   void CanDownload() {
     download_request_limiter_->CanDownloadImpl(
-        controller().tab_contents(), this);
+        controller().tab_contents(), -1, this);
     message_loop_.RunAllPending();
   }
 
   bool ShouldAllowDownload() {
     ask_allow_count_++;
     return allow_download_;
   }
 
  protected:
   class DownloadRequestLimiterTestDelegate
@@ skipping 145 matching lines @@
   ask_allow_count_ = continue_count_ = cancel_count_ = 0;
   CanDownload();
   ASSERT_EQ(0, ask_allow_count_);
   ASSERT_EQ(0, continue_count_);
   ASSERT_EQ(1, cancel_count_);
   // And the state shouldn't have changed.
   ASSERT_EQ(DownloadRequestLimiter::DOWNLOADS_NOT_ALLOWED,
             download_request_limiter_->GetDownloadStatus(
                 controller().tab_contents()));
 }