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

Unified Diff: components/autofill/content/renderer/autofill_agent.h

Issue 2766053002: [refactor] Fix autofill features for payments when the form is inside an OOPIF (Closed)
Patch Set: Move a method from WebWidgetClient to WebFrameClient Created 3 years, 9 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/content/renderer/autofill_agent.h
diff --git a/components/autofill/content/renderer/autofill_agent.h b/components/autofill/content/renderer/autofill_agent.h
index 698e5c9fc7095503c76b086d4e83ee42a96b3ffd..f5f11ef99dddc27473683ad2acf98ab277a47ab8 100644
--- a/components/autofill/content/renderer/autofill_agent.h
+++ b/components/autofill/content/renderer/autofill_agent.h
@@ -18,7 +18,6 @@
#include "components/autofill/content/renderer/form_cache.h"
#include "components/autofill/content/renderer/page_click_listener.h"
#include "content/public/renderer/render_frame_observer.h"
-#include "content/public/renderer/render_view_observer.h"
#include "mojo/public/cpp/bindings/binding.h"
#include "third_party/WebKit/public/web/WebAutofillClient.h"
#include "third_party/WebKit/public/web/WebFormControlElement.h"
@@ -96,29 +95,6 @@ class AutofillAgent : public content::RenderFrameObserver,
bool operator()(const FormData& lhs, const FormData& rhs) const;
};
- // Thunk class for RenderViewObserver methods that haven't yet been migrated
- // to RenderFrameObserver. Should eventually be removed.
- // http://crbug.com/433486
- class LegacyAutofillAgent : public content::RenderViewObserver {
- public:
- LegacyAutofillAgent(content::RenderView* render_view, AutofillAgent* agent);
- ~LegacyAutofillAgent() override;
-
- // Shuts the LegacyAutofillAgent down on RenderFrame deletion. Safe to call
- // multiple times.
- void Shutdown();
-
- private:
- // content::RenderViewObserver:
- void OnDestruct() override;
- void FocusChangeComplete() override;
-
- AutofillAgent* agent_;
-
- DISALLOW_COPY_AND_ASSIGN(LegacyAutofillAgent);
- };
- friend class LegacyAutofillAgent;
-
// Flags passed to ShowSuggestions.
struct ShowSuggestionsOptions {
// All fields are default initialized to false.
@@ -155,6 +131,7 @@ class AutofillAgent : public content::RenderFrameObserver,
void WillSubmitForm(const blink::WebFormElement& form) override;
void DidChangeScrollOffset() override;
void FocusedNodeChanged(const blink::WebNode& node) override;
+ void DidCompleteFocusChangeInFrame() override;
void OnDestruct() override;
// Fires IPC messages for a given form submission. Will always fire
@@ -170,10 +147,6 @@ class AutofillAgent : public content::RenderFrameObserver,
// times.
void Shutdown();
- // Pass-through from LegacyAutofillAgent. This correlates with the
- // RenderViewObserver method.
- void FocusChangeComplete();
-
// PageClickListener:
void FormControlElementClicked(const blink::WebFormControlElement& element,
bool was_focused) override;
@@ -260,9 +233,6 @@ class AutofillAgent : public content::RenderFrameObserver,
PasswordAutofillAgent* password_autofill_agent_; // Weak reference.
PasswordGenerationAgent* password_generation_agent_; // Weak reference.
- // Passes through RenderViewObserver methods to |this|.
- LegacyAutofillAgent legacy_;
-
// The ID of the last request sent for form field Autofill. Used to ignore
// out of date responses.
int autofill_query_id_;

Powered by Google App Engine
This is Rietveld 408576698