rAc: don't clobber user input when accepting autofill popup suggestion

chrome/browser/ui/autofill/autofill_dialog_view.h

Index: chrome/browser/ui/autofill/autofill_dialog_view.h
diff --git a/chrome/browser/ui/autofill/autofill_dialog_view.h b/chrome/browser/ui/autofill/autofill_dialog_view.h
index fdcbf6a8fd4b7048ea6c582612ec6d544aa5754c..279b4a96d6a14ecfbb89eeac0615ac9675b2af9c 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_view.h
+++ b/chrome/browser/ui/autofill/autofill_dialog_view.h
@@ -35,8 +35,10 @@ class AutofillDialogView {
   // Updates the button strip based on the current controller state.
   virtual void UpdateButtonStrip() = 0;
 
-  // Called when the contents of a section have changed.
-  virtual void UpdateSection(DialogSection section) = 0;
+  // Called when the contents of a section have changed. If |clear_user_input|,
+  // is true, the view should also clobber all manual inputs. Otherwise,
+  // previous user input should be respected.
+  virtual void UpdateSection(DialogSection section, bool clear_user_input) = 0;

[Ilya Sherman, 2013/04/04 03:59:55]

nit: If you keep this method, please use an enumerated constant rather than a
boolean for the type of the second variable.  As is, call sites are quite hard
to understand at a glance.

[Ilya Sherman, 2013/04/04 23:11:02]

^^^

[Evan Stade, 2013/04/05 23:24:01]

Done.

 
   // Fills |output| with data the user manually input.
   virtual void GetUserInput(DialogSection section, DetailOutputMap* output) = 0;