Some nits I found while looking around rAc() code.

chrome/browser/ui/autofill/autofill_dialog_types.cc

Index: chrome/browser/ui/autofill/autofill_dialog_types.cc
diff --git a/chrome/browser/ui/autofill/autofill_dialog_types.cc b/chrome/browser/ui/autofill/autofill_dialog_types.cc
index f6d6b07f791f3149b1e3529ca1a678e3b7cd85de..54631d48a2cff2cac0189d5b44d8b2750bf845f3 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_types.cc
+++ b/chrome/browser/ui/autofill/autofill_dialog_types.cc
@@ -8,7 +8,6 @@
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "grit/generated_resources.h"
-#include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace {
@@ -21,6 +20,23 @@ bool IsSureError(const autofill::ValidityMessage& message) {
 
 namespace autofill {
 
+#ifndef NDEBUG
+std::ostream& operator<<(std::ostream& out, DialogSection section) {

[Evan Stade, 2014/01/03 20:28:29]

I don't think this should be here.

[Dan Beam, 2014/01/03 20:31:04]

it's not in release builds

[groby-ooo-7-16, 2014/01/04 01:05:45]

See previous comment - I think it should be all or nothing. (See also
autofill::FormData, autofill::CreditCard for autofill examples)

+  switch (section) {
+    case SECTION_CC:
+      return out << "SECTION_CC";
+    case SECTION_BILLING:
+      return out << "SECTION_BILLING";
+    case SECTION_CC_BILLING:
+      return out << "SECTION_CC_BILLING";
+    case SECTION_SHIPPING:
+      return out << "SECTION_SHIPPING";
+  }
+  NOTREACHED();
+  return out;
+}
+#endif
+
 static const base::char16 kRangeSeparator = '|';
 
 DialogNotification::DialogNotification() : type_(NONE) {}