Cleanup various includes and namespace references.

content/browser/android/content_view_statics.cc

Index: content/browser/android/content_view_statics.cc
diff --git a/content/browser/android/content_view_statics.cc b/content/browser/android/content_view_statics.cc
index 31eaabeb0b52398e03805b9e207f7efcffc66998..1c4e9035d152413ba6670c9b5286a6cb5b1c80d3 100644
--- a/content/browser/android/content_view_statics.cc
+++ b/content/browser/android/content_view_statics.cc
@@ -83,7 +83,7 @@ class SuspendedProcessWatcher : public content::RenderProcessHostObserver {
     std::vector<int>::iterator pos = std::find(suspended_processes_.begin(),
                                                suspended_processes_.end(),
                                                host->GetID());
-    DCHECK_NE(pos, suspended_processes_.end());

[Sami, 2014/10/31 14:33:27]

Is this working around some bug in the DCHECK_NE macro? Could we fix that
instead?

[pasko, 2014/10/31 15:38:24]

I think *CHECK_NE macro (and similar) are better than their *CHECK(_ != _)
counterparts *only* by being able to print the values of the parameters in case
of mismatch, and only in cases we need the printing. We don't have good ways to
print iterators over collections (that I know of), so it seems better to _not_
do any useless printing (and instantiating the code to print) in this case.

In other words, not accepting CHECK_NE for iterators seems like a good feature
of the library (to me).

[Fabrice (no longer in Chrome), 2014/10/31 15:45:24]

I'm inclined to say the problem is here.
There is no conversion from std::vector<>::iterator to std::string when the
compiler tries to instantiate MakeCheckOpString in base/logging.h
DCHECK_OP is only supposed to be used for simple types, as I understand it.

+    DCHECK(pos != suspended_processes_.end());
     host->RemoveObserver(this);
     suspended_processes_.erase(pos);
   }