IWYU fixes for url/...

url/url_util.cc

Index: url/url_util.cc
diff --git a/url/url_util.cc b/url/url_util.cc
index cbb252f47132ff91643fba35dbcf338f13d8e160..0aa720cc0f8caa6847ada73fe3089dc93333b571 100644
--- a/url/url_util.cc
+++ b/url/url_util.cc
@@ -6,14 +6,17 @@
 
 #include <stddef.h>
 #include <string.h>
-#include <vector>
+#include <debug/vector>

[Łukasz Anforowicz, 2016/01/07 00:16:17]

I have no idea why IWYU thought that changing <vector> to <debug/vector> might
be a good idea...  I was building Debug flavor FWIW...

This is what iwyu.out said:
  ../../url/url_util.cc should add these lines:
  #include <debug/vector>                         // for vector

  ../../url/url_util.cc should remove these lines:
  - #include <vector>  // lines 10-10

Maybe this is happening because I built a debug flavor of clang/iwyu tool?

+#include <ostream>

[Łukasz Anforowicz, 2016/01/07 00:16:17]

I don't know why <ostream> is needed.  We do use << for streaming into a DCHECK
macro below, but 1) this should be covered by including base/logging.h + 2) we
don't use any stream-related classes.

FWIW, iwyu.out says:
  #include <ostream>  // for operator<<

 
 #include "base/debug/leak_annotations.h"
 #include "base/logging.h"
+#include "base/strings/string_piece.h"
 #include "base/strings/string_util.h"
+#include "url/third_party/mozilla/url_parse.h"
 #include "url/url_canon_internal.h"
-#include "url/url_file.h"
-#include "url/url_util_internal.h"
+#include "url/url_constants.h"
+#include "url/url_parse_internal.h"
 
 namespace url {