External autofill delegates.

content/browser/renderer_host/render_view_host.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 340 matching lines @@
 
   // Creates a full screen RenderWidget.
   void CreateNewFullscreenWidget(int route_id);
 
 #if defined(OS_MACOSX)
   // Select popup menu related methods (for external popup menus).
   void DidSelectPopupMenuItem(int selected_index);
   void DidCancelPopupMenu();
 #endif
 
+  // When using an external autofill delegate.
+  void SelectAutofillSuggestionAtIndex(int listIndex);

[Ilya Sherman, 2011/10/26 11:09:58]

We should not add a dependency on Autofill to content/.  Can this method be
defined on the AutofillExternalDelegate itself; or otherwise on the
AutofillManager class?

[John Grabowski, 2011/10/27 03:05:59]

Yes.  Looks like I didn't finish the Carnitas-ification.
I took a stab at doing this the right way but would appreciate feedback.

[Ilya Sherman, 2011/10/27 11:01:55]

Thanks, the Carnitas-ification looks good with the latest patch set :)

+
   void ToggleSpeechInput();
 
   void set_save_accessibility_tree_for_testing(bool save) {
     save_accessibility_tree_for_testing_ = save;
   }
 
   const webkit_glue::WebAccessibility& accessibility_tree_for_testing() {
     return accessibility_tree_;
   }
 
@@ skipping 261 matching lines @@
   // The termination status of the last render view that terminated.
   base::TerminationStatus render_view_termination_status_;
 
   // A list of observers that filter messages.  Weak references.
   ObserverList<RenderViewHostObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderViewHost);
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_H_