iwyu: Include stringprintf.h where appropriate, part 1.

base/json/string_escape.cc

Index: base/json/string_escape.cc
diff --git a/base/json/string_escape.cc b/base/json/string_escape.cc
index 0b12439603d019143351c086f6dcc9dbb16a0752..b4415b8fa3b951d748fb305c553ba29a53305446 100644
--- a/base/json/string_escape.cc
+++ b/base/json/string_escape.cc
@@ -6,6 +6,7 @@
 
 #include <string>
 
+#include "base/stringprintf.h"
 #include "base/string_util.h"
 
 namespace base {
@@ -63,7 +64,7 @@ void JsonDoubleQuoteT(const STR& str,
         // 2. Technically, we could also pass through c > 126 as UTF8, but this
         //    is also optional.  It would also be a pain to implement here.
         unsigned int as_uint = static_cast<unsigned int>(c);
-        StringAppendF(dst, "\\u%04X", as_uint);
+        base::StringAppendF(dst, "\\u%04X", as_uint);

[Nico, 2011/05/10 02:57:51]

hm, these are not necessary because of ADL. does iwyu add these? do we prefer
explicitness over adl? it's a bit less flexible.

[James Hawkins, 2011/05/10 04:01:43]

No, I added them.  I prefer them, but I don't think the style guide says
anything about this. My rationale is that the style guide bans the use of using
entire namespaces (specifically because of ambiguity), and in my mind relyingon
ADL to get this right is even worse than using namespace base at the top of the
file.

       } else {
         unsigned char ascii = static_cast<unsigned char>(*it);
         dst->push_back(ascii);