predictors: Replace deprecated TestBrowserThread by TestBrowserThreadBundle

chrome/browser/predictors/resource_prefetch_predictor_test_util.h

Index: chrome/browser/predictors/resource_prefetch_predictor_test_util.h
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_test_util.h b/chrome/browser/predictors/resource_prefetch_predictor_test_util.h
index a86beb9bb97f4fc701ba40cb0196feccebb31221..daa3bd397f27125b1c077832b9c904dca48b441e 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_test_util.h
+++ b/chrome/browser/predictors/resource_prefetch_predictor_test_util.h
@@ -33,10 +33,10 @@ RedirectData CreateRedirectData(const std::string& primary_key,
                                 uint64_t last_visit_time = 0);
 
 // For printing failures nicely.
-void PrintTo(const PrefetchData& data, ::std::ostream* os);
-void PrintTo(const ResourceData& resource, ::std::ostream* os);
-void PrintTo(const RedirectData& data, ::std::ostream* os);
-void PrintTo(const RedirectStat& redirect, ::std::ostream* os);
+std::ostream& operator<<(std::ostream& stream, const PrefetchData& data);

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

that's neat, I did not know it was possible to use both  operator<< and PrintTo
in googletest

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

Yeah, I think that it makes sense to use PrintTo only if there is already
decleared some operator<< for type T. In other cases << is more convenient.
For more information:
https://github.com/google/googletest/blob/master/googletest/docs/AdvancedGuid...

+std::ostream& operator<<(std::ostream& stream, const ResourceData& resource);
+std::ostream& operator<<(std::ostream& stream, const RedirectData& data);
+std::ostream& operator<<(std::ostream& stream, const RedirectStat& redirect);
 
 bool operator==(const PrefetchData& lhs, const PrefetchData& rhs);
 bool operator==(const ResourceData& lhs, const ResourceData& rhs);