predictors: Replace deprecated TestBrowserThread by TestBrowserThreadBundle

chrome/browser/predictors/resource_prefetch_predictor_test_util.cc

Index: chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc b/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
index 3920d883bd45162926013b565bf4d31d5bad2116..e7e7b18439eb52fc7e38149440f84eb48a93e840 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
+++ b/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
@@ -57,37 +57,34 @@ RedirectData CreateRedirectData(const std::string& primary_key,
   return data;
 }
 
-void PrintTo(const PrefetchData& data, ::std::ostream* os) {
-  *os << "[" << data.primary_key() << "," << data.last_visit_time() << "]\n";
-  for (const ResourceData& resource : data.resources()) {
-    *os << "\t\t";
-    PrintTo(resource, os);
-    *os << "\n";
-  }
+std::ostream& operator<<(std::ostream& os, const PrefetchData& data) {
+  os << "[" << data.primary_key() << "," << data.last_visit_time() << "]\n";
+  for (const ResourceData& resource : data.resources())
+    os << "\t\t" << resource << "\n";

[pasko, 2016/10/17 14:46:42]

nit: std::endl instead of '\n' for extra pretty on hypothetical Windows?

[alexilin, 2016/10/17 15:32:35]

Uh, I've thought that std::endl is kinda undesirable thing. I've heard that it's
slower than simple '\n'.

[pasko, 2016/10/17 16:09:19]

TIL:
1. std::endl also does flushing of the buffer
2. it would output '\r\n' on Windows only if the stream is a file that is open
in text mode.

So: std::endl sounds better in tests, where portability and flushing messages
ASAP is more important than speed.

+  return os;
 }
 
-void PrintTo(const ResourceData& resource, ::std::ostream* os) {
-  *os << "[" << resource.resource_url() << "," << resource.resource_type()
-      << "," << resource.number_of_hits() << "," << resource.number_of_misses()
-      << "," << resource.consecutive_misses() << ","
-      << resource.average_position() << "," << resource.priority() << ","
-      << resource.has_validators() << "," << resource.always_revalidate()
-      << "]";
+std::ostream& operator<<(std::ostream& os, const ResourceData& resource) {
+  return os << "[" << resource.resource_url() << "," << resource.resource_type()
+            << "," << resource.number_of_hits() << ","
+            << resource.number_of_misses() << ","
+            << resource.consecutive_misses() << ","
+            << resource.average_position() << "," << resource.priority() << ","
+            << resource.has_validators() << "," << resource.always_revalidate()
+            << "]";
 }
 
-void PrintTo(const RedirectData& data, ::std::ostream* os) {
-  *os << "[" << data.primary_key() << "," << data.last_visit_time() << "]\n";
-  for (const RedirectStat& redirect : data.redirect_endpoints()) {
-    *os << "\t\t";
-    PrintTo(redirect, os);
-    *os << "\n";
-  }
+std::ostream& operator<<(std::ostream& os, const RedirectData& data) {
+  os << "[" << data.primary_key() << "," << data.last_visit_time() << "]\n";
+  for (const RedirectStat& redirect : data.redirect_endpoints())
+    os << "\t\t" << redirect << os;

[alexilin, 2016/10/17 15:32:35]

oh, wow

[pasko, 2016/10/17 16:09:19]

:)

+  return os;
 }
 
-void PrintTo(const RedirectStat& redirect, ::std::ostream* os) {
-  *os << "[" << redirect.url() << "," << redirect.number_of_hits() << ","
-      << redirect.number_of_misses() << "," << redirect.consecutive_misses()
-      << "]";
+std::ostream& operator<<(std::ostream& os, const RedirectStat& redirect) {
+  return os << "[" << redirect.url() << "," << redirect.number_of_hits() << ","
+            << redirect.number_of_misses() << ","
+            << redirect.consecutive_misses() << "]";
 }
 
 bool operator==(const PrefetchData& lhs, const PrefetchData& rhs) {