Reduce number of unnamed-type-template-args violations.

chrome/browser/net/predictor_unittest.cc

Index: chrome/browser/net/predictor_unittest.cc
===================================================================
--- chrome/browser/net/predictor_unittest.cc	(revision 97489)
+++ chrome/browser/net/predictor_unittest.cc	(working copy)
@@ -263,12 +263,12 @@
 
 // Return a motivation_list if we can find one for the given motivating_host (or
 // NULL if a match is not found).
-static ListValue* FindSerializationMotivation(
-    const GURL& motivation, const ListValue& referral_list) {
+static ListValue* FindSerializationMotivation(const GURL& motivation,
+                                              const ListValue& referral_list) {

[Nico, 2011/08/19 20:34:23]

This indentation style is strictly worse than the previous one (if you rename
the function, you now have to reindent the param on the following name), so I'd
revert this change

[brettw, 2011/08/21 05:37:03]

I usually recommend the style applied by the patch:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Functi...
(Align params to after the ( if they fit, use 4 spaces if they don't.)

   CHECK_LT(0u, referral_list.GetSize());  // Room for version.
   int format_version = -1;
   CHECK(referral_list.GetInteger(0, &format_version));
-  CHECK_EQ(Predictor::PREDICTOR_REFERRER_VERSION, format_version);
+  CHECK_EQ(Predictor::kPredictorReferrerVersion, format_version);
   ListValue* motivation_list(NULL);
   for (size_t i = 1; i < referral_list.GetSize(); ++i) {
     referral_list.GetList(i, &motivation_list);
@@ -284,7 +284,7 @@
 static ListValue* NewEmptySerializationList() {
   base::ListValue* list = new base::ListValue;
   list->Append(
-      new base::FundamentalValue(Predictor::PREDICTOR_REFERRER_VERSION));
+      new base::FundamentalValue(Predictor::kPredictorReferrerVersion));
   return list;
 }