Setting up external delegate calls to allow a future delegate to handle autofill

chrome/browser/autofill/autofill_manager.h

Index: chrome/browser/autofill/autofill_manager.h
diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h
index 4115187595fbdc0b1ffb6f53b0d90c6dae36f418..6ddd7add1cf123169a9dbadbd50eb01ebd8bc359 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -154,6 +154,14 @@ class AutofillManager : public TabContentsObserver,
   void OnDidPreviewAutofillFormData();
   void OnDidFillAutofillFormData(const base::TimeTicks& timestamp);
   void OnDidShowAutofillSuggestions(bool is_new_popup);
+  void OnShowAutofillSuggestions(
+      int query_id,
+      const std::vector<string16>& autofill_values,
+      const std::vector<string16>& autofill_labels,
+      const std::vector<string16>& autofill_icons,
+      const std::vector<int>& autofill_unique_ids);

[Ilya Sherman, 2011/11/08 21:06:16]

nit: I think this method is no longer used -- right?

[csharp, 2011/11/09 16:18:37]

Correct and removed.

On 2011/11/08 21:06:16, Ilya Sherman wrote:

+  void OnSetAutofillElementBounds(int x, int y, int width, int height);

[John Grabowski, 2011/11/08 20:46:02]

Label coordinates as screen or document relative.

[csharp, 2011/11/09 16:18:37]

No longer used, but I added comment to new function that gets gfx::Rect to say
that it is document-relative.

On 2011/11/08 20:46:02, John Grabowski wrote:

+  void OnHideAutofillPopup();
 
   // Fills |host| with the RenderViewHost for this tab.
   // Returns false if Autofill is disabled or if the host is unavailable.