Clean-up coding style

chrome/browser/safe_browsing/malware_details_cache.cc

 // Copyright (c) 2012 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.
 //
 // Implementation of the MalwareDetails class.
 
 #include "chrome/browser/safe_browsing/malware_details.h"
 
 #include "base/bind.h"
 #include "base/lazy_instance.h"
@@ skipping 49 matching lines @@
   return has_started_;
 }
 
 MalwareDetailsCacheCollector::~MalwareDetailsCacheCollector() {}
 
 // Fetch a URL and advance to the next one when done.
 void MalwareDetailsCacheCollector::OpenEntry() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   DVLOG(1) << "OpenEntry";
 
-  if (resources_it_ == resources_->end()) { // We are done.
+  if (resources_it_ == resources_->end()) {  // We are done.

[Scott Hess - ex-Googler, 2014/06/04 16:19:08]

Or just remove the comment.  Being at the end and calling "AllDone(true)" seems
to imply that things are done.

[jaekyeom, 2014/06/05 11:58:57]

Done.

     AllDone(true);
     return;
   }
 
   if (!request_context_getter_.get()) {
     DVLOG(1) << "Missing request context getter";
     AllDone(false);
     return;
   }
 
@@ skipping 116 matching lines @@
       base::Bind(&MalwareDetailsCacheCollector::OpenEntry, this));
 }
 
 void MalwareDetailsCacheCollector::AllDone(bool success) {
   DVLOG(1) << "AllDone";
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   *result_ = success;
   BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, callback_);
   callback_.Reset();
 }