Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(704)

Side by Side Diff: net/ftp/ftp_network_transaction.cc

Issue 149013: Add DCHECKs to the states that should be entered only when the... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2008 The Chromium Authors. All rights reserved. Use of this 1 // Copyright (c) 2008 The Chromium Authors. All rights reserved. Use of this
2 // source code is governed by a BSD-style license that can be found in the 2 // source code is governed by a BSD-style license that can be found in the
3 // LICENSE file. 3 // LICENSE file.
4 4
5 #include "net/ftp/ftp_network_transaction.h" 5 #include "net/ftp/ftp_network_transaction.h"
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/string_util.h" 8 #include "base/string_util.h"
9 #include "net/base/connection_type_histograms.h" 9 #include "net/base/connection_type_histograms.h"
10 #include "net/base/net_errors.h" 10 #include "net/base/net_errors.h"
(...skipping 221 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 rv = DoCtrlResolveHostComplete(rv); 232 rv = DoCtrlResolveHostComplete(rv);
233 break; 233 break;
234 case STATE_CTRL_CONNECT: 234 case STATE_CTRL_CONNECT:
235 DCHECK(rv == OK); 235 DCHECK(rv == OK);
236 rv = DoCtrlConnect(); 236 rv = DoCtrlConnect();
237 break; 237 break;
238 case STATE_CTRL_CONNECT_COMPLETE: 238 case STATE_CTRL_CONNECT_COMPLETE:
239 rv = DoCtrlConnectComplete(rv); 239 rv = DoCtrlConnectComplete(rv);
240 break; 240 break;
241 case STATE_CTRL_READ: 241 case STATE_CTRL_READ:
242 DCHECK(rv == OK);
242 rv = DoCtrlRead(); 243 rv = DoCtrlRead();
243 break; 244 break;
244 case STATE_CTRL_READ_COMPLETE: 245 case STATE_CTRL_READ_COMPLETE:
245 rv = DoCtrlReadComplete(rv); 246 rv = DoCtrlReadComplete(rv);
246 break; 247 break;
247 case STATE_CTRL_WRITE_USER: 248 case STATE_CTRL_WRITE_USER:
248 DCHECK(rv == OK); 249 DCHECK(rv == OK);
249 rv = DoCtrlWriteUSER(); 250 rv = DoCtrlWriteUSER();
250 break; 251 break;
251 case STATE_CTRL_WRITE_PASS: 252 case STATE_CTRL_WRITE_PASS:
(...skipping 22 matching lines...) Expand all
274 break; 275 break;
275 case STATE_CTRL_WRITE_RETR: 276 case STATE_CTRL_WRITE_RETR:
276 DCHECK(rv == OK); 277 DCHECK(rv == OK);
277 rv = DoCtrlWriteRETR(); 278 rv = DoCtrlWriteRETR();
278 break; 279 break;
279 case STATE_CTRL_WRITE_SIZE: 280 case STATE_CTRL_WRITE_SIZE:
280 DCHECK(rv == OK); 281 DCHECK(rv == OK);
281 rv = DoCtrlWriteSIZE(); 282 rv = DoCtrlWriteSIZE();
282 break; 283 break;
283 case STATE_CTRL_WRITE_CWD: 284 case STATE_CTRL_WRITE_CWD:
285 DCHECK(rv == OK);
284 rv = DoCtrlWriteCWD(); 286 rv = DoCtrlWriteCWD();
285 break; 287 break;
286 case STATE_CTRL_WRITE_LIST: 288 case STATE_CTRL_WRITE_LIST:
289 DCHECK(rv == OK);
287 rv = DoCtrlWriteLIST(); 290 rv = DoCtrlWriteLIST();
288 break; 291 break;
289 case STATE_CTRL_WRITE_MDTM: 292 case STATE_CTRL_WRITE_MDTM:
293 DCHECK(rv == OK);
290 rv = DoCtrlWriteMDTM(); 294 rv = DoCtrlWriteMDTM();
291 break; 295 break;
292 case STATE_CTRL_WRITE_QUIT: 296 case STATE_CTRL_WRITE_QUIT:
297 DCHECK(rv == OK);
293 rv = DoCtrlWriteQUIT(); 298 rv = DoCtrlWriteQUIT();
294 break; 299 break;
295 300
296 case STATE_DATA_RESOLVE_HOST: 301 case STATE_DATA_RESOLVE_HOST:
302 DCHECK(rv == OK);
297 rv = DoDataResolveHost(); 303 rv = DoDataResolveHost();
298 break; 304 break;
299 case STATE_DATA_RESOLVE_HOST_COMPLETE: 305 case STATE_DATA_RESOLVE_HOST_COMPLETE:
300 rv = DoDataResolveHostComplete(rv); 306 rv = DoDataResolveHostComplete(rv);
301 break; 307 break;
302 case STATE_DATA_CONNECT: 308 case STATE_DATA_CONNECT:
303 DCHECK(rv == OK); 309 DCHECK(rv == OK);
304 rv = DoDataConnect(); 310 rv = DoDataConnect();
305 break; 311 break;
306 case STATE_DATA_CONNECT_COMPLETE: 312 case STATE_DATA_CONNECT_COMPLETE:
307 rv = DoDataConnectComplete(rv); 313 rv = DoDataConnectComplete(rv);
308 break; 314 break;
309 case STATE_DATA_READ: 315 case STATE_DATA_READ:
316 DCHECK(rv == OK);
310 rv = DoDataRead(); 317 rv = DoDataRead();
311 break; 318 break;
312 case STATE_DATA_READ_COMPLETE: 319 case STATE_DATA_READ_COMPLETE:
313 rv = DoDataReadComplete(rv); 320 rv = DoDataReadComplete(rv);
314 break; 321 break;
315 default: 322 default:
316 NOTREACHED() << "bad state"; 323 NOTREACHED() << "bad state";
317 rv = ERR_FAILED; 324 rv = ERR_FAILED;
318 break; 325 break;
319 } 326 }
(...skipping 555 matching lines...) Expand 10 before | Expand all | Expand 10 after
875 &io_callback_); 882 &io_callback_);
876 } 883 }
877 884
878 int FtpNetworkTransaction::DoDataReadComplete(int result) { 885 int FtpNetworkTransaction::DoDataReadComplete(int result) {
879 DLOG(INFO) << read_data_buf_->data(); // The read_data_buf_ is NULL 886 DLOG(INFO) << read_data_buf_->data(); // The read_data_buf_ is NULL
880 // terminated string. 887 // terminated string.
881 return result; 888 return result;
882 } 889 }
883 890
884 } // namespace net 891 } // namespace net
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698