rAc: minimal crash fix for bug 342990.

chrome/browser/ui/views/autofill/autofill_dialog_views.cc

Index: chrome/browser/ui/views/autofill/autofill_dialog_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
index 827468a8067e3b1580d21846ad9c2f79db6fd5c0..001c05ea0c2e5f02405730239230ba8d323ca367 100644
--- a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
@@ -2210,6 +2210,10 @@ void AutofillDialogViews::InputEditedOrActivated(ServerFieldType type,
   DecoratedTextfield* decorated = TextfieldForType(type);
   views::Combobox* combobox = ComboboxForType(type);
   DCHECK_NE(!!combobox, !!decorated);
+
+  if (!combobox && !decorated)

[Evan Stade, 2014/02/14 22:26:11]

this is fine, but please put the DCHECK_NE below this if, and put a comment that
says something like "Both views may be null if the event may come from an
inactive section, which may occur when using an IME."

[Dan Beam, 2014/02/14 22:47:35]

Done.

+    return;
+
   DetailsGroup* group = decorated ? GroupForView(decorated) :
                                     GroupForView(combobox);
   base::string16 text = decorated ?