Comment #6 on issue 18036 by [email protected]: Undisable New FTP  
tests in url_request_unittest.cc
http://code.google.com/p/chromium/issues/detail?id=18036

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=22752

------------------------------------------------------------------------
r22752 | [email protected] | 2009-08-07 12:26:00 -0700 (Fri, 07 Aug  
2009) | 8 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/url_request/url_request_new_ftp_job.cc?r1=22752&r2=22751
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/url_request/url_request_unittest.cc?r1=22752&r2=22751

Re-enable remaining FTP unit tests.

Also, use NotifyDone in URLRequestNewFtpJob instead of NotifyStartError.  
This way is_pending in URLRequest is correctly reset to false.

TEST=Covered by net_unittests.
http://crbug.com/18036

Review URL: http://codereview.chromium.org/165142
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to