[net/base] Forward declaration of classes instead of including header

net/base/directory_lister.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.
 
 #include "net/base/directory_lister.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/file_util.h"
 #include "base/i18n/file_util_icu.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"

[mmenke, 2015/05/19 14:45:16]

While you're here, I believe you can remove this header - doesn't look like
MessageLoop is currently used in this file.

[Pranay, 2015/05/20 03:09:35]

This file has FROM_HERE flag ( dependency - base/location.h) which is included
in message_loop.h that's why we will need to include location.h after removing
messsage_loop.h ..Done

+#include "base/message_loop/message_loop_proxy.h"
 #include "base/threading/thread_restrictions.h"
 #include "base/threading/worker_pool.h"
 #include "net/base/net_errors.h"
 
 namespace net {
 
 namespace {
 
 bool IsDotDot(const base::FilePath& path) {
   return FILE_PATH_LITERAL("..") == path.BaseName().value();
@@ skipping 169 matching lines @@
 
 void DirectoryLister::OnListFile(const DirectoryListerData& data) {
   delegate_->OnListFile(data);
 }
 
 void DirectoryLister::OnListDone(int error) {
   delegate_->OnListDone(error);
 }
 
 }  // namespace net