Remove unused include in sequenced_worker_pool.h

content/browser/loader/upload_data_stream_builder.cc

Index: content/browser/loader/upload_data_stream_builder.cc
diff --git a/content/browser/loader/upload_data_stream_builder.cc b/content/browser/loader/upload_data_stream_builder.cc
index f3d79c6fd5a96fab01db0e9f76ad53a4e9e06936..389a8e9b39425a76b661b801011136ddd6a7e222 100644
--- a/content/browser/loader/upload_data_stream_builder.cc
+++ b/content/browser/loader/upload_data_stream_builder.cc
@@ -14,6 +14,7 @@
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
+#include "base/single_thread_task_runner.h"

[Charlie Harrison, 2016/11/01 02:53:35]

I'm not sure why this can't be a fwd-decl. There's no use of
scoped_refptr<SingleThreadTaskRunner> in the header.

[gab, 2016/11/01 13:30:58]

It's subtle but there is, in UploadDataStreamBuilder::Build():
|file_task_runner| is passed into FileElementReader's constructor. The former is
a SingleThreadTaskRunner* while the latter takes a TaskRunner* as argument. In
order to perform the implicit upcast, the compiler has to know that
SingleThreadTaskRunner implements TaskRunner.

 #include "content/browser/fileapi/upload_file_system_file_element_reader.h"
 #include "content/common/resource_request_body_impl.h"
 #include "net/base/elements_upload_data_stream.h"
@@ -24,6 +25,10 @@
 #include "storage/browser/blob/blob_storage_context.h"
 #include "storage/browser/blob/upload_blob_element_reader.h"
 
+namespace base {
+class TaskRunner;
+}
+
 namespace disk_cache {
 class Entry;
 }