Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(203)

Unified Diff: components/autofill/core/browser/autofill_driver.h

Issue 17572015: Begin abstracting sending of IPC from autofill core code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Response to review, add tests Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/autofill/core/browser/autofill_driver.h
diff --git a/components/autofill/core/browser/autofill_driver.h b/components/autofill/core/browser/autofill_driver.h
index 178e31b98b83682392d6e0760b691028eedac94f..f67887da16a269130dc175d5d044f570630162e3 100644
--- a/components/autofill/core/browser/autofill_driver.h
+++ b/components/autofill/core/browser/autofill_driver.h
@@ -5,14 +5,17 @@
#ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_DRIVER_H_
#define COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_DRIVER_H_
+#include "components/autofill/core/common/form_data.h"
+
namespace content {
class WebContents;
}
namespace autofill {
-// Interface that provides access to the driver-level context in which Autofill
-// is operating. A concrete implementation must be provided by the driver.
+// Interface that allows Autofill core code to interact with its driver (i.e.,
+// obtain information from it and give information to it). A concrete
+// implementation must be provided by the driver.
class AutofillDriver {
public:
virtual ~AutofillDriver() {}
@@ -20,6 +23,14 @@ class AutofillDriver {
// TODO(blundell): Remove this method once shared code no longer needs to
// know about WebContents.
virtual content::WebContents* GetWebContents() = 0;
+
+ // Returns true iff the renderer is available for communication.
+ virtual bool RendererIsAvailable() = 0;
+
+ // Forwards |data| to the renderer. |query_id| is the id of the renderer's
+ // original request for the data. Is a no-op if the renderer is not currently
Ilya Sherman 2013/06/26 22:54:58 nit: "Is a no-op" -> "This method is a no-op"
blundell 2013/06/27 21:58:27 Done.
+ // available.
+ virtual void SendFormDataToRenderer(int query_id, const FormData& data) = 0;
};
} // namespace autofill

Powered by Google App Engine
This is Rietveld 408576698